GenDeathrow / MoreChickens

5 stars 14 forks source link

nickelchicken collides with lime chicken #57

Open scones opened 5 years ago

scones commented 5 years ago

the lime chicken has the breeding of: white + green set here (2 years ago): https://github.com/setycz/ChickensMod/blob/03497fec4a08f156f8b448db76316985f5fe0ebd/src/main/java/com/setycz/chickens/ChickensMod.java

whereas the nickelchicken has the breeding of: white + green

i'd suggest another combo, like silver + green to make it work.

i am no java programmer tho, nor a modder. thus the "fix" might be incomplete. I am just assuming this is the correct place.

best regards

drewburr commented 5 years ago

@scones Making the Nickel chicken dependent on the Silver chicken would turn the Nickel chicken into a tier 4, where it's currently a tier 2. The silver chicken requires white + gray and is a tier 3, so perhaps nickel could be green + gray? This would make nickel a tier 3, and prevent dependents from shifting tiers as well

scones commented 5 years ago

does the tier matter?

drewburr commented 5 years ago

Tier determines speed of breeding and resource generation. Really the only reason why I commented at all is because really this isn't a problem. Getting a 2nd chicken isn't a big deal unless you're trying to automate the process. Keeping the nickel chicken below tier 4 prevents changing the tiers of other chickens that require it.

As a toss-back, is silver a must? I feel that we should be keep change as minimal as possible, especially for something that's more for convenience rather than a fix or balance.

Just a side note, Nether Quartz isn't a bad option either.

scones commented 5 years ago

Well, it is a fix. You can't breed nickel with the current combination. If i find some time, i will change it to nether quartz. that should keep the tier.

GenDeathrow commented 5 years ago

I will check the changes and make a determination as well. ty