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

Galactic effects & correcting new planet enviornmental effects #71

Merged
merged 6 commits into from
Nov 29, 2024

Conversation

CrosswaveOmega
Copy link
Contributor

This Pull Request contains the known effects of all observed Active Planet Effects, switches biome keys to the internal game names for the sake of consistency, and corrections for the temperature alternating planets.

@chatterchats
Copy link
Collaborator

Regarding the planets.json - While acknowledging that out data is also potentially mistaken, how are you determining which evnironmental effects are on the the planet? For Example, Klen Dath II (planet_001) has both intense heat and extreme cold on your version. (This is in addition to the fact that the only environmental effect it should have is Sandstorm as per the helldiversData repo by Shalzuth)

@CrosswaveOmega
Copy link
Contributor Author

CrosswaveOmega commented Nov 15, 2024

@chatterchats
One of the patches on September 17th (patch 01.001.100) internally changed the way the game handles temperature. It's now specific to the weather settings instead of the environmental effects.

For example, on Klen Dath II and planets that share it's settings, the extreme heat/cold modifiers is now applied whenever the time in the mission area is day/night respectively. I built a special script to get this mapped.

Copy link

@lambstream lambstream left a comment

Choose a reason for hiding this comment

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

Please update the coding style. You're using spaces instead of tabs.

@chatterchats
Copy link
Collaborator

@chatterchats One of the patches on September 17th (patch 01.001.100) internally changed the way the game handles temperature. It's now specific to the weather settings instead of the environmental effects.

For example, on Klen Dath II and planets that share it's settings, the extreme heat/cold modifiers is now applied whenever the time in the mission area is day/night respectively. I built a special script to get this mapped.

@CrosswaveOmega
Hm... That change may indicate we need to restructure, since that field has pretty much always correlated to the planet's gameplay_modifiers. I'll talk w/ the other folks about this to decide how we want to move forward with that

Just a brief heads-up, the json repo is formatted using 4-width tabs and not spaces, if you could please get that corrected ^^

@CrosswaveOmega
Copy link
Contributor Author

I am attempting to get the formatting resolved in my editor, hopefully this corrects it.

@chatterchats
Copy link
Collaborator

I am attempting to get the formatting resolved in my editor, hopefully this corrects it.

Looks a lot better!, Thank you

Copy link
Member

@dealloc dealloc left a comment

Choose a reason for hiding this comment

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

@chatterchats requesting your opinion here, but would sorting the json files by key alphabetically be feasible?
would make future PR's/diffs a little easier to maintain as now things seem to jump around between updates

planets/biomes.json Show resolved Hide resolved
planets/environmentals.json Outdated Show resolved Hide resolved
planets/planets.json Outdated Show resolved Hide resolved
weather_effects can be renamed to something else if needed.
@chatterchats chatterchats requested a review from dealloc November 29, 2024 16:04
@dealloc dealloc merged commit 38e1c1c into helldivers-2:master Nov 29, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants