Skip to content
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

Rework Generators 2.0 ™️ #2255

Merged
merged 8 commits into from
Dec 7, 2023

Conversation

ghzdude
Copy link
Contributor

@ghzdude ghzdude commented Dec 5, 2023

What

Fixes large combustion engines not boosting on initial recipe run.
Prevents single-block generators from overclocking
overclock button for single-block generators are gone 🦀
Fixes #2246

Implementation Details

the lubricant and oxygen checks have been moved to more appropriate places in the recipe logic. where i moved them in #2229 wasn't actually a good place for them, since canRecipeProgress() is only checked once every second.
lubricant check is moved to shouldSearchForRecipes(), the oxygen is moved to update()

FuelRecipeLogic returns false for overclocking, forcing them to parallelize

Outcome

large combustion engines check if they are boosted before actually running the recipe
single-block gens are no longer sus

@ghzdude ghzdude requested a review from a team as a code owner December 5, 2023 22:13
@ghzdude ghzdude added the type: bug Something isn't working label Dec 5, 2023
@ghzdude ghzdude changed the title Fix Large Combustion Engine not Boosting Correctly Rework Generators 2.0 ™️ Dec 5, 2023
@ghzdude ghzdude force-pushed the gh-fix-multi-gens-2 branch from 582e7cd to 8d9af15 Compare December 5, 2023 23:24
@serenibyss serenibyss enabled auto-merge (squash) December 7, 2023 02:51
@serenibyss serenibyss merged commit 3f8d6f1 into GregTechCEu:master Dec 7, 2023
4 checks passed
serenibyss pushed a commit that referenced this pull request Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Critical flaw in generator code
2 participants