Tweaks to Growing Plant Render

[Xefyr0]

Cross-mod Integration

No response

Minecraft Version

1.20.1 Forge

Feature Description

After testing out some plants with the growth render, I noticed that while most plants grow beautifully & intuitively, some plants are a bit odd.

Torchflowers, Pumpkins, and Melons don't appear to fully mature during the recipe cycle - Torchflowers never get their distinctive fiery hues and Melons/Pumpkins never display their signature blocks.
This is because these plants use a separate block to represent their fully-grown states - if another growth mode could be made where this separate block is tacked on the end of the "Growing Plant" growth mode as a final growth stage then that should lead to the most intuitive results.

Pitcher Plants use the "Double Translate" growth mode since they're a 2-block tall plant but they have multiple age stages so the only thing that grows up from the ground is the Age 0 bulb. It should use the "Growing Plant" growth mode instead.

Pink Petals use the "Translate" growth mode, but they have multiple block states with different flower amounts. In-game, it looks like nothing happens for most of the recipe time then three tiny pink flowers appear toward the end.
Instead, it should operate like Sea Pickles.

Chorus Flowers/Plants and Cocoa Beans grow particularly weirdly - Chorus Flowers grow in a double stack when the bottom of the stack should be a Chorus Plant block, and the Cocoa Beans grow floating instead of attaching to any available blocks around them.
Unlike the previous cases, these would likely require substantially more effort to implement for less benefit - I would not blame you for skipping these and just doing the others above.

[RubenVerg]

I think it'd be useful to investigate whether these can be detected generically or if they have to be special cased; the latter would be unfortunate for similar blocks added by other mods.

[RubenVerg]

Torchflowers: they work by actually supporting an age that is bigger than the age property suggests, changing the block state into the one of the full plant. If mods follow this same pattern, will work with them as well.
Pumpkins and Melons: these are instances of StemBlock which nicely has a getFruit() method that returns the result block. If mods also use StemBlock, will work with them as well.
Pitcher Plants: both has an age property and a half property. just need to do the has age property check before the has half property check, and then allow for double-tall aging blocks in the aging renderer. If mods do it the same way vanilla does, will work with them as well. Do note that Monifactory greenhouses use the actual pitcher plant, and not the pitcher pods, and so will use the double-translate mode. Changing the recipe honestly seems like a better idea even outside this particular necessity.
Pink Petals: use the flower_amount property, which is functionally equal to the pickles one except for name. It is likely that mods will create their own properties with different names for things like this, but there is probably no reasonable way to support them all.
Chorus Flowers: as a side effect of the pitcher plant fix, they now render as aging blocks; which makes them be the white version most of the time and then become the purple one near the end. Possibly better than how it was before.
Cocoa Beans: Being able to search for physical blocks during rendering very likely requires restructuring everything.