-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Combine last_rot_check and last_temp_check variables #38370
Closed
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Would these changes affect or resolve issue #38353 in any way? |
Most likely not. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Please do not 'bump' or comment on this issue unless you are actively working on it. Stale issues, and stale issues that are closed are still considered. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
SUMMARY: Infrastructure "Combine last_rot_check and last_temp_check variables"
Purpose of change
last_rot_check
andlast_temp_check
are almost alway identical. And the situations where they are different can be changed so that they are same.So there is no need for them to be separate.
Describe the solution
The time at which temperature and rot was last checked is now tracked by
last_temp_check
.Other code changes that came with it:
process
toprocess_temperature_rot
.process_temperature_rot
toprocess
.set_item_temperature
no longer resets last_temp_check. Process the item before setting temperature or usereset_temp_check
if rot doesn't need calculating.When food is heated up it will skip one rot calculation cycle (up to 10 minutes).
When liguids are combined in container they will skip one rot calculation cyle (up to 10 minutes).
I do not think these are enough of an issue to warrant workaround. It is also not easy to process items directly because the item does not know where it is (in something like freezer?).
Testing
Items rot at same rate as before.
Cooked items have same rot as in current version. Tested recipes without heating, with heating and multiple input materials.
Additional context