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

Craftable chest freezer #73762

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

Conversation

XtraButtery
Copy link

Summary

Balance "Adds a crafting recipe for chest freezers"

Purpose of change

Adds a crafting recipe for chest freezers. Chest freezers were replaced by the current household freezers, and are in the most reductive terms just a household freezer turned onto its side, so it makes sense to be able to craft these, given all components are readily accessible.

Describe the solution

Added in the recipe. Requirements are based upon the relative surface area between the chest freezer and the household freezer. Chest freezer has a surface area of roughly +50% so several components have had their requirements increased by 50% for this recipe.

Describe alternatives you've considered

Leaving the recipe out entirely.

Amending the crafting recipe to use welding proficiency. I still think this might be useful, but as this is my first contribution I wanted to keep it simple stuip

Testing

Confirmed that the recipe was accessible in game using debug to give myself the required skills, books and components.

Additional context

It may be useful to recipe audit refrigeration appliances at some point, as the recipes seem to be a little odd compared to the results. This seemed like too large of a scope for me right now, but many refrigeration appliances have been changed by #73048 and the recipes might need to be reworked as a result.

Confirmed recipe is working, increased the crafting requirements.

Chest freezer has a surface area roughly 50% greater than the household freezer, so I increased the requirements by about 50% for structural materials.
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Crafting / Construction / Recipes Includes: Uncrafting / Disassembling Appliance/Power Grid Anything to do with appliances and power grid Game: Balance Balancing of (existing) in-game features. new contributor labels May 14, 2024
{ "id": "WRENCH", "level": 1 }
],
"components": [
[ [ "sheet_metal", 30 ] ],
Copy link
Contributor

@Karol1223 Karol1223 May 14, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Something went horrendously wrong here. This is 180kg alone, a chest freezer is 50kg. It's also almost 15 m2, so are you sure the surface area here is right?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As mentioned under additional context I think all refrigeration needs a wider audit as I don't think any of them are currently correct.

I based surface area off of the relative increase between the household freezer and the chest freezer, rather than the surface area of the sheet metal.

I could amend this recipe to be 10 sheet metal, which would be roughly accurate to the ~4.571m^2 surface area of the chest freezer, but then it would use the same amount of sheet metal as a minifreezer and half that of a household freezer.

Given that I wasn't really sure what I was doing with this pull request (I've never submitted a pull request on github before) I wanted to Keep It Simple Stupid for myself, to make sure I knew how to make a submission before going back in later and fixing all the recipes and adding in the rest of the missing recipes, which seemed like the best option for me as a first-time contributor.

You're absolutely correct though, the recipes are pretty wrong right now, but the issue came before my PR.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As mentioned under additional context I think all refrigeration needs a wider audit as I don't think any of them are currently correct.

They were just added, and Guardian cited their sources in JSON comments. I don't think they're blatantly incorrect, because I know Guardian spent weeks hunting the numbers down. The weight does seem suspiciously low, but they're definitely not all incorrect.

You're absolutely correct though, the recipes are pretty wrong right now, but the issue came before my PR.

Yeah, but you shouldn't make the issue worse by adding more to it. I don't expect you to audit all fridge recipes, but don't copy over their mistakes. Please make sure the weight at least adds up. We really desperately need to stop normalizing the violation of laws of physics in recipes...

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They were just added, and Guardian cited their sources in JSON comments. I don't think they're blatantly incorrect, because I know Guardian spent weeks hunting the numbers down. The weight does seem suspiciously low, but they're definitely not all incorrect.

I should clarify I meant that the recipes were incorrect not the specs on the fridges and freezers.

If they're going to be craftable there is an argument for adjusting the weight of some of these, as I imagine they're probably not using the same sheet metal we are to achieve some of these weights, but that's a compromise on reality that I'm not sure can easily be squared away unless we add in a "makeshift freezer" variant on all freezers.

Yeah, but you shouldn't make the issue worse by adding more to it. I don't expect you to audit all fridge recipes, but don't copy over their mistakes. Please make sure the weight at least adds up. We really desperately need to stop normalizing the violation of laws of physics in recipes...

Fair point. as I said, I was trying to opt for consistency, before going in and fixing all the recipes with a second PR once I was more confident, but I could quickly fix this one instead, though it will end up being wildly unbalanced in the short term.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reverted to draft while I resolve the recipe

@XtraButtery XtraButtery marked this pull request as draft May 14, 2024 16:25
@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Appliance/Power Grid Anything to do with appliances and power grid 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 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 new contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants