-
-
Notifications
You must be signed in to change notification settings - Fork 509
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add ramping heat to the Thermalily #4665
base: 1.20.x
Are you sure you want to change the base?
Conversation
…f lava is provided promptly
…he Thermalily's cooldown is based on the normal distribution
So, to summarize the intended changes in this PR:
Static automation that always assumes the worst-case of 5 minutes cooldown is unaffected by this change, producing just under 1/3rd of a pool per hour. Manually feeding the flower may be slightly less efficient, since the average cooldown time at heat 0 is somewhat higher than before the change. Dynamic automation that tracks the actual cooldown time can be much more efficient at a bit over 2.5 pools per hour. That's in the general area of a Gourmaryllis alternating between cooked cod and salmon every 3 seconds (where the difficulty is obtaining that much fish for extended run times), or a single-flower 100-cycle 6-block Dandelifeon setup. |
@@ -70,7 +58,7 @@ public void tickFlower() { | |||
ticksSinceFueled++; | |||
} else if (burnTime == startBurnTime) { | |||
if (ticksSinceFueled <= FAST_PROVIDE_TICKS) { | |||
heat = heat < 10 ? heat + 1 : heat; | |||
heat = heat < MAX_HEAT ? heat + 1 : heat; | |||
} else { | |||
heat = 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I can tell, this is the only place where heat
is reset. If lava is provided before the end of the cooldown, the cooldown restarts (as intended), but heat does not reset. I'm not sure if that's a bad thing, as lava is still wasted, so fine by me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ideally I would have made it so that providing lava before the end of the cooldown would cause the thermalily to "burn" into a dead bush, but that would require either changing the logic in FluidGeneratingFlower (doable, requires some refactoring and moving of code), or use the call to getCooldownTime(false), (janky and prone to errors if other mods/addons/code calls that).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think just resetting the heat is an ok penalty here
One can wonder whether this goes against the spirit of the thermalily, if it's considered a manual "quick, get me some mana" flower, kinda like the rosa arcana. But, since we already added the variable cooldown as an automation puzzle, i think this is a reasonable extension of that idea. I'll review the code itself more in-depth later. |
We also need documentation. And an effect showing that the flower is "heated up" would be cool, if that's not already a thing |
Slight (or, massive, actually) issue with being able to take lava from cauldrons: It effectively turns the thermalily into a passive flower |
True, but giving lava at the wrong time (which the passive use variant would inevitably do) makes it inefficient. A cauldron refills from pointed dripstone roughly every 19.4 minutes. And if you put multiples near the flower, you risk it taking the lava during the cooldown, which restarts the cooldown. IMHO it's the best way to automate the Thermalily, unless we can provide another way for Botania to make lava renewable without player interaction. |
That amounts to ~83500 mana per hour, vs a hydroangea's 72000. You could just place a row of thermalilies and have them work better than hydroangeas, except they would never decay. Setup and flower cost is a bit more expensive than hydroangeas (earth + fire rune, and you need the lava and dripstone), but this PR would effectively turn the thermalily into a slightly more expensive, slightly better, and never-decaying version of the hydroangeas. I'm starting to think the original random cooldown change was a bad idea. The Thermalily Was Not Meant To Be Automated, and you can't really make it automatable without completely breaking the mod at this point. |
Not going to argue with that. That modification broke the effort-to-output "rule" of generating flowers by being way too slow for the amount of effort the automation required. |
This was the original reason behind wanting to make the thermalily "burn" into a dead bush if you supply lava too early (as opposed to just losing heat if too late), as a fully passive setup will eventually cause the thermalily to burn. |
Currently, the Thermalily breaks a fundamental design principle of Botania, as automating it even partially is quite complex while only giving fairly paltry amounts of mana. This PR aims to fix that by decreasing the average cooldown of the Thermalily if lava is provided promptly after the cooldown ends. This stacks up to 10 times, eventually reducing the average cooldown down to 27 seconds. This means if the player is able to supply a large amount of lava consistently, the Thermalily will produce almost thrice its current mana/second.
Potential additional changes:
Allow the Thermalily to consume lava from caldrons, allowing for player-less automation
Increase the mana per tick rather than or in addition to the cooldown reduction to improve the mana per lava bucket
Implement a stronger penalty for "mistiming" a heated Thermalily (explodes, catches on fire, decays, etc.)