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

Deconstructing a water heater yields a 100L tank instead of 60L #68359

Merged
merged 2 commits into from
Sep 28, 2023

Conversation

db48x
Copy link
Contributor

@db48x db48x commented Sep 27, 2023

Summary

Balance "Deconstructing a water heater yields a 100L tank instead of 60L (plus the usual bits and bobs)."

Purpose of change

Fixes #68316

Describe the solution

I did a quick search for water heaters on the website of the nearest home improvement store and found that the Rheem model XE36S06ST45U0 is a good match for the stats of the existing household_water_heater item at 240L exterior volume and 54kg mass. See
https://images.thdstatic.com/catalog/pdfImages/2d/2d7ed116-1a8c-439d-b7fe-a62a0a98f806.pdf for published dimensions.

As this water heater has an internal capacity of 36 US gallons (136L), I changed the deconstruction recipe to yield a 100L tank instead of the 60L it used to be. I also changed the keg_capacity of the heater to match.

Describe alternatives you've considered

I originally intended to use the 200L 55gal_drum for this change, primarily because I myself have a 50–gallon heater. However, I noticed that this didn’t leave a lot of room for insulation, and went looking for specifications on real water heaters.

Of course, this size of water heater is intended for use in small apartments; larger houses would have larger heaters (up to 100 gallons even). I’m resisting the temptation to add another size or two just on general principle, but I note that mansions do exist.

Testing

I tested this by disassembling a water heater and noting that I got a 100L tank as intended. Later I noticed that the keg_capacity was off; turns out that it's in cups not liters.

Additional context

Screenshot 2023-09-27 at 15-54-19 2d7ed116-1a8c-439d-b7fe-a62a0a98f806 pdf

Screenshot from 2023-09-27 15-51-58

plus the usual bits and bobs.

I did a quick search for water heaters on the website of the nearest
home improvement store and found that the Rheem model XE36S06ST45U0 is
a good match for the stats of the existing household_water_heater item
at 240L exterior volume and 54kg mass. See
<https://images.thdstatic.com/catalog/pdfImages/2d/2d7ed116-1a8c-439d-b7fe-a62a0a98f806.pdf>
for published dimensions.

As this water heater has an internal capacity of 36 US gallons (136L),
I changed the deconstruction recipe to yield a 100L tank instead of
the 60L it used to be. I also changed the keg_capacity of the heater
to match.

Of course, this size of water heater is intended for use in small
apartments; larger houses would have larger heaters (up to 100 gallons
even). I’m resisting the temptation to add another size or two just on
general principle, but I note that mansions do exist.

Fixes CleverRaven#68316
@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) new contributor labels Sep 27, 2023
@GuardianDll
Copy link
Member

> I’m resisting the temptation to add another size or two

Just add it, i don't see any reason to not

@db48x
Copy link
Contributor Author

db48x commented Sep 27, 2023

To do that well I would want to update the mapgen for the mansion and perhaps some of the larger houses, but that’s a lot more than I want to do tonight. :)

@GuardianDll
Copy link
Member

Sure. Fyi mapgen allows to define variants of the same terrain/furniture, so for most places you can make simple replacement "@": "f_water_heater" -> "@": [ [ "f_water_heater", 7 ], [ "f_water_heater_big", 3] ] so the game would generate 70% of default water heaters, and 30% of big water heaters, or something similar

@db48x
Copy link
Contributor Author

db48x commented Sep 27, 2023

Yes, I did something similar for a different patch I’m working on (unrelated to water heaters).

so 100 liters needs to be entered as 400, not 100. Oops.
@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 Sep 28, 2023
@Maleclypse Maleclypse merged commit 9bace41 into CleverRaven:master Sep 28, 2023
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 Fields / Furniture / Terrain / Traps Objects that are part of the map or its 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.

larger tank for water heaters
3 participants