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

yet another portable charcoal kiln audit #78622

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

marilynias
Copy link
Contributor

Summary

Balance "make charcoal kiln place-able"

Purpose of change

while tackling #78619 I found that the char_kiln implementation was kind of a mess where each step has its own item, you need to craft to fill the kiln with wood and uncraft to receive the final product, while the furniture (with its own independent construction recipe) had a lot more intuitive kilning process. This PR seeks to unite them.
resolves #78619.

Describe the solution

  • replace the construction ingredients of the furniture kiln with the char_kiln item and adjust requirements
  • add deploy action to char_kiln
  • obsolete the 3 step-items and their recipes

Describe alternatives you've considered

Only fixing lit kiln size. As seen in the issue, but there will be mistakes/missed items in the future.

Testing

Craft, deploy, deconstruct kiln.
Load save with all 4 stages of the kiln to ensure migration works. (will loose the wood/charcoal inside)

Additional context

The process for f_kiln_metal_empty is all in c++, maybe its possible to EOC'ify it?

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Crafting / Construction / Recipes Includes: Uncrafting / Disassembling Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. <Bugfix> This is a fix for a bug (or closes open issue) Game: Balance Balancing of (existing) in-game features. astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Dec 17, 2024
@IdleSol
Copy link
Contributor

IdleSol commented Dec 17, 2024

Pay attention. These are different kilns. The stationary one is 250 liters. That's why it's made of 4 tanks. You're using a 60-liter "mobile" kiln.

And if I remember correctly, they have different material to coal conversion ratios.

Anyway, I'm against this PR in its current form.

UPD.

@marilynias
Copy link
Contributor Author

Pay attention. These are different kilns. The stationary one is 250 liters. That's why it's made of 4 tanks. You're using a 60-liter "mobile" kiln.

And if I remember correctly, they have different material to coal conversion ratios.

thanks, quick opinion check;
Is it desired to have 2 different metal kilns? (one small one big), in which case ill revert to just fixing the original issue.
If not, which one should be the default one? Since there are no reference models, ill reference
CharcoalProduction.pdf p8-15.
the conversion rates are very similar (~30% vs 28-30%).

@IdleSol
Copy link
Contributor

IdleSol commented Dec 18, 2024

My personal opinion. There must be two versions:

  1. A stationary one with high efficiency
  2. Mobile, with less efficiency

The way it is now. I base it on the fact that we have mobile and stationary smokehouses. And if you have a mobile base, it's more convenient for you to carry the mobile versions.

"id": "metal_smoking_rack",


and

As for changes. You can just add new furniture ~ f_kiln_metal_empty_small or f_kiln_metal_empty_mobile or whatever. I'm bad at English. Then the mobile would be set on the ground before use. Which will give you what you want.

@marilynias marilynias marked this pull request as draft December 18, 2024 11:14
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Dec 18, 2024
@marilynias marilynias changed the title make metal charcoal kiln furniture craftable from charcoal kiln item yet another portable charcoal kiln audit Dec 19, 2024
@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` and removed astyled astyled PR, label is assigned by github actions labels Dec 19, 2024
@IdleSol
Copy link
Contributor

IdleSol commented Dec 19, 2024

The volume of the item (char_kiln) is 201 liters. And if you place it (f_kiln_portable_empty), it has a volume of 250 liters. I may be wrong, but this 250 liters is the internal volume.

It's supposed to be 200 liters? But I'm not sure it is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Crafting / Construction / Recipes Includes: Uncrafting / Disassembling Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. 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.

Lit charcoal kilns are are tiny compare to it's other stage (experimental).
2 participants