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

Reign in carbide craft quantity #77964

Merged
merged 1 commit into from
Nov 21, 2024
Merged

Conversation

kevingranade
Copy link
Member

@kevingranade kevingranade commented Nov 18, 2024

Summary

None

Purpose of change

@EvanBalster pointed out to me that the batch size for this recipe set by #76788 was way too large.
On review I either neglected to account for charges in the inputs or double counted or something, because it's off by about 100x.

Describe the solution

Drop the inputs, outputs, and charges required by 100x.

Testing

Re-tested in game and found that the amount needed for the reaction is about half a liter of carbide mix instead of 50, and that it yields about a "metric cup" of carbide.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Crafting / Construction / Recipes Includes: Uncrafting / Disassembling json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Nov 18, 2024
@EvanBalster
Copy link
Contributor

This change brings the recipe down to something that could reasonably fit inside a furnace made of 40 8-centimeter firebricks. Previously, creating enough premix for one smelting would take some 575 hours of crafting; that's now reduced to about 7 hours.

Carbide premix, black powder and various other mineral powders currently must be crafted 'by hand' using a mortar and pestle, being too hard to put into a food processor. I wonder if an industrial grinding/mixing apparatus such as a ball mill might be a sensible addition to the game in the future?

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Nov 21, 2024
@Night-Pryanik Night-Pryanik merged commit 679b0be into master Nov 21, 2024
21 of 27 checks passed
@kevingranade kevingranade deleted the kevingranade-not-so-much-carbide branch November 21, 2024 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions Crafting / Construction / Recipes Includes: Uncrafting / Disassembling [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants