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

Tweaked canning ratios for fruit, tomatoes, and liver #38226

Merged
merged 3 commits into from
Feb 22, 2020

Conversation

jkraybill
Copy link
Contributor

Summary

SUMMARY: Content "Tweaked canning ratios for fruit, tomatoes, and liver"

Purpose of change

Fixes #37504, Fixes #38201

There were issues with canning recipes producing under- or over-filled cans/jars for fruit, tomatoes, and liver. There were some calorie/ingredient inconsistencies with canned fruit.

Describe the solution

Fixed a few issues:

  • standardized canned fruit to use 1 unit of sugar_standard instead of differing amounts of only sugar, and standardized ingredients so different sizes of canned fruit produce serving sizes of the same caloric values

  • fixed jarred/canned fruit and tomatoes to produce 24/24 3L jars, 4/4 jars and 2/2 cans instead of 12/24, 2/4, and 1/2

  • fixed canned liver to produce 4/4 cans instead of 8/4

  • standardized fruit & tomato canning to use (Fruit / 2 + Canning_Water) amount of water

  • renamed "opened large tin can" to "large opened tin can" to be consistent with "small opened tin can" and "medium opened tin can"

Describe alternatives you've considered

Holding off until a larger adjustment to food preservation in the next release.

Testing

Produced the 3 can sizes before and after the changes for tomatoes, fruit, and liver.

Additional context

There are still some inconsistencies with other canning recipes, but to avoid scope creep I didn't touch recipes outside of fruit, tomatoes, and liver. I plan on doing a larger sweep in the next release.

@Rivet-the-Zombie Rivet-the-Zombie merged commit cbd994a into CleverRaven:master Feb 22, 2020
@jkraybill
Copy link
Contributor Author

May want to check that the comestible tests pass -- they failed earlier because of the calorie counts changing, so I tweaked them.

@jkraybill jkraybill deleted the 37504 branch March 1, 2020 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants