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

Auditing portable charcoal kiln #77139

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AudBobb
Copy link
Contributor

@AudBobb AudBobb commented Oct 18, 2024

Summary

Balance "Audit portable charcoal kiln"

Purpose of change

Resolve #76658 and remove pocket dimensions from portable charcoal kilns

Describe the solution

Mostly implemented as per #76658.

  1. Changed recipe for char_kiln to use a 60L tank instead of a 2L tank
  2. Adjusted recipe for the filled charcoal kiln to use 50L of whatever materials are being used, from whatever arbitrary amounts where there before, they were all over the place. I chose 50L instead of 60 because this allows to balance the output to the 50L log, instead of having one recipe for the log, and one for the smaller items that can fit up to 60L. This is open for change. I also removed pine boughs from the possible ingredients, because pine needles aren't going to turn into charcoal, they are just going to burn up entirely. Thought about removing bone splinters as well, but I don't know enough about that to say one way or the other.
  3. Change the output to give charcoal(5000) instead of charcoal(10000). This equates to a conversion factor of 0.5, slightly less than the 0.6 factor for the lower tier stationary charcoal kiln. I believe this makes sense, as a portable version should logically be less efficient than a stationary version could be. There will be less insulation, less heat, just all around worse.

Describe alternatives you've considered

Removing it, but I don't think that's really necessary. Also considered leaving it at 2L and adjusting everything to that, but that would leave the item practically useless.

Testing

Loaded into test world, checked recipes

Additional context

@github-actions github-actions bot added <Bugfix> This is a fix for a bug (or closes open issue) Game: Balance Balancing of (existing) in-game features. [JSON] Changes (can be) made in JSON Crafting / Construction / Recipes Includes: Uncrafting / Disassembling astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Oct 18, 2024
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 <Bugfix> This is a fix for a bug (or closes open issue) Crafting / Construction / Recipes Includes: Uncrafting / Disassembling Game: Balance Balancing of (existing) in-game features. [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.

Tool check: charcoal kiln
1 participant