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

Balance dilute fruit juice crafting time #39662

Merged
merged 2 commits into from
Apr 18, 2020

Conversation

Drewscriver
Copy link
Contributor

Summary

SUMMARY: Balance "Reduce crafting time of 50% fruit juice compared to 100% fruit juice"

Purpose of change

Reduce crafting time of 50% fruit juice (squeezing 1 fruit + 1 water) compared to 100% fruit juice (squeezing 2 fruits) to reflect the reduced amount of effort.

Describe the solution

Both recipes took 5 minutes. Reduced the 50% fruit recipe to 3 minutes to reflect less effort (but still some prep time).

Describe alternatives you've considered

Removing the 50% fruit juice recipe, or implementing a 'dilute fruit juice' item in it's place.

Testing

Made some fruit juice. Crafting times were as expected.

Additional context

In the process of testing, I found some bugs in how fruit juice is handled. When unloading liquid from one container to another, the final mix takes on the value of the liquid it is poured into, rather than an average value. So someone could make 14 units of 50% juice and pour it into a gallon jug with 1 unit of 100% juice, and end up with a full gallon of 100% juice. Likewise, strawberry juice could become cherry juice, etc.

The two recipes for fruit juice took the same amount of time to craft, even though one was 100% juice, and the other 50% juice and 50% water.  This change cuts the 50% water recipe crafting time from 5 minutes to 3 minutes to reflect the reduced amount of effort.
@kevingranade kevingranade merged commit a93e935 into CleverRaven:master Apr 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants