ModTweaker

ModTweaker

88M Downloads

[Request] Move Centrifuge Chances

Yulife opened this issue ยท 8 comments

commented

For @jaredlll08
Right now it is...

public static void addRecipe(int timePerItem, IItemStack itemInput, int[] chances, IItemStack[] output)

However, the chances should be after the Output.

commented

Could you please just set chance afterwards? We'll take care of unification at a very late point. If we do it now, people's script gonna break and that will cause many issues.

commented

Going to overrule here, Yulife knows his shit (I know it may seen like he doesn't but he really does...) So Yulife wins on this one

commented

@Voidi could you take care of that?

commented

agree, but even better:
input, timePerItem, output[], chances[]

commented

Nah, time is at the beginning. That is with all other different recipes in Modtweaker.

commented

Nope Blast Furnace for example http://minetweaker3.powerofbytes.com/wiki/ModTweaker:Railcraft_Support

We should think about some general rules.

I like the format Minetweaker has: first the output and then ingredients.
Vice versa is okay to.

commented

Yulife: You don't need to worry about that. When if/you standardise it. You can still keep the old methods. (Then remove them later down the line).

I think I pretty much just went with what the mods themselves were using? xD

commented

Yeah, I used your documentation as a template :)