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

Remove guaranteed residential solar, add random residential solar #75227

Merged
merged 21 commits into from
Jul 29, 2024

Conversation

Maleclypse
Copy link
Member

Summary

Balance "Remove guaranteed residential solar, add random residential solar"

Purpose of change

Some house designs had guaranteed solar panels on the roof. No more. Adjust solar panel installations to Massachusetts ratio.

Describe the solution

I went and looked up the total number of single family homes in MA and then looked up the total number of residential solar installations in MA. 1,515,125 vs 177,143 This comes to 11.7%. Since the game covers all of New England and I didn't want to do weights in fractions of a percent I averaged most houses to have an 11% chance of spawning a solar panel nest. I also removed all guaranteed solar panel spawns on housetops.

Describe alternatives you've considered

Nope

Testing

Additional context

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display Game: Balance Balancing of (existing) in-game features. labels Jul 25, 2024
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

[JSON & C++ formatters](https://github.com/CleverRaven/Cataclysm-DDA/blob/master/doc/DEVELOPER_TOOLING.md)

[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"nested": {
"1": {
"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 4 ],
[ "null", 88 ]
]
}
}


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"object": {
"mapgensize": [ 2, 2 ],
"place_furniture": { "furn": "f_solar_unit", "x": [ 0, 1 ], "y": [ 0, 1 ] }
}


[JSON & C++ formatters] reported by reviewdog 🐶

"object": {
"mapgensize": [ 1, 2 ],
"place_furniture": { "furn": "f_solar_unit", "x": 0, "y": [ 0, 1 ] }
}


[JSON & C++ formatters] reported by reviewdog 🐶

"object": {
"mapgensize": [ 1, 3 ],
"place_furniture": { "furn": "f_solar_unit", "x": 0, "y": [ 0, 2 ] }
}


[JSON & C++ formatters] reported by reviewdog 🐶

"object": {
"mapgensize": [ 3, 3 ],
"place_furniture": { "furn": "f_solar_unit", "x": [ 0, 2 ], "y": [ 0, 2 ] }
}

@github-actions github-actions bot added the astyled astyled PR, label is assigned by github actions label Jul 25, 2024
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

[JSON & C++ formatters](https://github.com/CleverRaven/Cataclysm-DDA/blob/master/doc/DEVELOPER_TOOLING.md)

[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"nested": {
"1": {
"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 4 ],
[ "null", 88 ]
]
}
}


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"chunks": [
[ "residential_1x2_solar", 6 ],
[ "residential_2x2_solar", 2 ],
[ "residential_1x3_solar", 3 ],
[ "null", 88 ]
]


[JSON & C++ formatters] reported by reviewdog 🐶

"object": {
"mapgensize": [ 2, 2 ],
"place_furniture": { "furn": "f_solar_unit", "x": [ 0, 1 ], "y": [ 0, 1 ] }
}


[JSON & C++ formatters] reported by reviewdog 🐶

"object": {
"mapgensize": [ 1, 2 ],
"place_furniture": { "furn": "f_solar_unit", "x": 0, "y": [ 0, 1 ] }
}


[JSON & C++ formatters] reported by reviewdog 🐶

"object": {
"mapgensize": [ 1, 3 ],
"place_furniture": { "furn": "f_solar_unit", "x": 0, "y": [ 0, 2 ] }
}


[JSON & C++ formatters] reported by reviewdog 🐶

"object": {
"mapgensize": [ 3, 3 ],
"place_furniture": { "furn": "f_solar_unit", "x": [ 0, 2 ], "y": [ 0, 2 ] }
}

@github-actions github-actions bot added the json-styled JSON lint passed, label assigned by github actions label Jul 25, 2024
@GuardianDll
Copy link
Member

GuardianDll commented Jul 25, 2024

is there a reason 3x3 solar is used exactly once?
is there a reason to make [ "residential_1x2_solar", 6 ], [ "residential_2x2_solar", 2 ], [ "residential_1x3_solar", 3 ], [ "null", 88 ] 100 times, instead of making a single nest with it
and, lastly, is there a reason to not put it in the roof_palette instead of adding it manually?

i'll fix it, because i feel it would be better this way

UPD: only the first question is what i would like to know, if possible

@Maleclypse
Copy link
Member Author

Maleclypse commented Jul 25, 2024

is there a reason 3x3 solar is used exactly once? is there a reason to make [ "residential_1x2_solar", 6 ], [ "residential_2x2_solar", 2 ], [ "residential_1x3_solar", 3 ], [ "null", 88 ] 100 times, instead of making a single nest with it and, lastly, is there a reason to not put it in the roof_palette instead of adding it manually?

i'll fix it, because i feel it would be better this way

UPD: only the first question is what i would like to know, if possible

So the 3x3 is replacing a 3x3 spawn on exactly one fairly large house. I was expecting there to be multiple houses with 3x3 spawns when going through but only found one and a 4x3 that really didn't seem to fit the size of the house. The answer to the rest of your questions is "I often stop looking for better ways to do a thing once I've found a way to do it, because this is how I avoid getting locked in decision paralysis of weighing the options." :) Thank you for improving this.

edit: I knew there would be more missing Ms in mapgen and they may need additional nests placed but I'll take care of hunting that down.

@ehughsbaird
Copy link
Contributor

I looked at this a while ago and got some different stats, from https://www.eia.gov/todayinenergy/detail.php?id=54379

  • 4.7% of Northeast homes have solar
  • 2.5% of commercial buildings

But the reason I was looking at it was because the monocrystalline solar panels (solar_panel_v2) only spawn one place in game, on a car. So I found some stats on the production of those year by year and found that if the solar added by year matched the ratios of monocrystalline/polycrystalline production (and assuming solar from before stats start is negligible), there'd be about 34% monocrystalline panels by power production (but it takes more polycrystalline panels to produce the same amount as a monocrystalline panel, but the stats are for production of panels, so...).

Year Solar Added (MW) Monosilicon (%) Added Monosilicon (MW)
2011 42.65 25 10.66
2012 134.5 25 33.63
2013 226.94 25 56.74
2014 269.11 25 67.28
2015 286.53 25 71.63
2016 394 25 98.5
2017 571 35 199.85
2018 370 45 166.5
2019 237 65 154.05
Total 2531.73 - 858.83

858.83/2531.73 = 34% high-efficiency monosilicon panels

Production Percent from https://www.ise.fraunhofer.de/content/dam/ise/de/documents/publications/studies/Photovoltaics-Report.pdf
Solar Added from https://www.mass.gov/info-details/renewable-energy-snapshot#installed-solar-capacity-in-massachusetts-

It's obviously not great data and I don't won't to add more work to you, but if you're interested it seems like it could fit in this PR.

@Maleclypse
Copy link
Member Author

Maleclypse commented Jul 25, 2024

I looked at this a while ago and got some different stats, from https://www.eia.gov/todayinenergy/detail.php?id=54379

* 4.7% of Northeast homes have solar

* 2.5% of commercial buildings

But the reason I was looking at it was because the monocrystalline solar panels (solar_panel_v2) only spawn one place in game, on a car. So I found some stats on the production of those year by year and found that if the solar added by year matched the ratios of monocrystalline/polycrystalline production (and assuming solar from before stats start is negligible), there'd be about 34% monocrystalline panels by power production (but it takes more polycrystalline panels to produce the same amount as a monocrystalline panel, but the stats are for production of panels, so...).
Year Solar Added (MW) Monosilicon (%) Added Monosilicon (MW)
2011 42.65 25 10.66
2012 134.5 25 33.63
2013 226.94 25 56.74
2014 269.11 25 67.28
2015 286.53 25 71.63
2016 394 25 98.5
2017 571 35 199.85
2018 370 45 166.5
2019 237 65 154.05
Total 2531.73 - 858.83

858.83/2531.73 = 34% high-efficiency monosilicon panels

Production Percent from https://www.ise.fraunhofer.de/content/dam/ise/de/documents/publications/studies/Photovoltaics-Report.pdf Solar Added from https://www.mass.gov/info-details/renewable-energy-snapshot#installed-solar-capacity-in-massachusetts-

It's obviously not great data and I don't won't to add more work to you, but if you're interested it seems like it could fit in this PR.

I'm guessing part of the difference in percentages comes from multi unit homes vs single unit homes. Since in MA single family homes are about 1/2 57% per https://framinghamsource.com/index.php/2022/01/16/57-2-of-massachusetts-homes-are-single-family-2nd-lowest-in-u-s/ the number of multi unit homes and there are few incentives for renters or landlords to add solar to a multifamily I think my math meshes with your math. I'd have to create an advanced solar furniture which would be scope creep for this PR but definitely is something to consider for a followup. This is already touching too many files in my opinion to add a different set of files to look at as well. :)

Edit: your source had 100k installations in 2020 and mine had 178k in I think 2023 I'll look back if you think I should shrink the chances in the nest?

@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions and removed json-styled JSON lint passed, label assigned by github actions labels Jul 25, 2024
data/json/mapgen/irradiator_1.json Outdated Show resolved Hide resolved
data/json/mapgen/shelter.json Outdated Show resolved Hide resolved
data/json/mapgen/shelter.json Outdated Show resolved Hide resolved
data/json/mapgen/shelter.json Outdated Show resolved Hide resolved
data/json/mapgen/zoo.json Outdated Show resolved Hide resolved
data/json/mapgen/zoo.json Outdated Show resolved Hide resolved
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Jul 27, 2024
@github-actions github-actions bot added BasicBuildPassed This PR builds correctly, label assigned by github actions and removed BasicBuildPassed This PR builds correctly, label assigned by github actions labels Jul 27, 2024
@dseguin dseguin merged commit 1f31d54 into CleverRaven:master Jul 29, 2024
23 of 24 checks passed
@Maleclypse Maleclypse deleted the Solar-Roof-nest-for-houses branch September 7, 2024 01:39
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 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 Map / Mapgen Overmap, Mapgen, Map extras, Map display
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants