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

Batch scaling for water, soups, cooking meats, smoking #9807

Merged
merged 5 commits into from Nov 4, 2014

Conversation

Projects
None yet
4 participants
@boydkr
Copy link
Contributor

commented Nov 2, 2014

I figured we should probably showcase the recipe batch time scaling change that went in recently.

  • Set-up time for boiling water ~10-15 min, adjusted recipes for clean water, broths, and soups
  • Set-up time for smoking anything is ~1hr
  • Cooked meat recipes adjusted with additional ~10 of setup time
  • Recipes for boiling off all water for salt/sugar had been tweaked for batch savings, but not as drastically as water/soups

@boydkr boydkr changed the title Batch scaling Batch scaling for water, soups, cooking meats, smoking Nov 2, 2014

@boydkr

This comment has been minimized.

Copy link
Contributor Author

commented Nov 2, 2014

@Rivet-the-Zombie

This comment has been minimized.

Copy link
Member

commented Nov 2, 2014

Nice!

@KA101

This comment has been minimized.

Copy link
Contributor

commented Nov 2, 2014

I figured we should probably showcase the recipe batch time scaling change

First thing folks are gonna notice is that we've just slapped the two most commonly run recipes in the game with a big ol' time nerf. Agreed that it's more realistic and encourages the code's use, but I don't think it draws attention to batching and the time savings therein.

You might consider adding a message when crafting batch-enabled recipes saying You prepare a %s batch of %r. where s is an adjective derived from the saved time (small = inefficient, medium = some time saved, large = you're approaching or hit the max time saved) and r is the recipe.

@Rivet-the-Zombie

This comment has been minimized.

Copy link
Member

commented Nov 2, 2014

They're nerfed if you're working on the small scale, but improved greatly if you're working with a bunch at a time. I like this PR.

@boydkr

This comment has been minimized.

Copy link
Contributor Author

commented Nov 2, 2014

I agree that it is a nerf for clean water and cooked meat for small scale production.

For clean, water, the time is basically the same as today, plus 6 minutes, if you're working with batches of 4 or more.

Batch size Incremental time (min) Cumulative time
1 5 5
2 2.5 7.5
3 1.4 8.8
4 1.1 9.9
N > 4 1 N+6

For cooked meat, it's effectively the same as today plus 17 min for batches of 5 or more.

I'm completely open to balance discussions.

As far as adding a nice message describing the efficiency of the recipe you just completed (or better yet, in the recipe selection and/or batch selection menu before you start crafting), I'm all for that as well, but it's definitely outside my current level of familiarity with the UI / message system for right now. I may take that on next, but not as a part of this PR

@@ -1028,6 +1040,7 @@
"time": 18000,

This comment has been minimized.

Copy link
@boydkr

boydkr Nov 2, 2014

Author Contributor

I missed this one, need to change to ~60min to match regular bologna recipe

@boydkr boydkr force-pushed the boydkr:batch_scaling branch to d397123 Nov 3, 2014

@Robik81

This comment has been minimized.

Copy link
Contributor

commented Nov 3, 2014

Just be careful not to make small batches time cost too prohibitive for perishables (i.e. initial time should not be too extreme for these).

I like this.

@KA101

This comment has been minimized.

Copy link
Contributor

commented Nov 4, 2014

I posted about this in Latest Experimental. Hopefully folks will see the HOWTO.

@KA101 KA101 self-assigned this Nov 4, 2014

@KA101 KA101 merged commit 6841dcd into CleverRaven:master Nov 4, 2014

1 check passed

default
Details

@boydkr boydkr deleted the boydkr:batch_scaling branch Feb 13, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.