-
Notifications
You must be signed in to change notification settings - Fork 548
Revert atmos presets #38438
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
Revert atmos presets #38438
Conversation
Thank you, I owe you my life |
why not just change the cac calculations? |
lmfao |
actually, a better solution would be to only revert the changes to max1. resolves the pressure change issue, and keeps high pressures (which are safe) still being marked as safe |
to what exactly? The original stuff worked fine for everything (even snaxi) until the preset change which as far as I'm aware was only made for Jungle as a workaround to pressurisation issues. Jungle is not in rotation, are you asking me to re-engineer other code to preserve a workaround for a map that isn't in rotation? This fix is to address issues that are impacting rounds currently.
While I agree that reverting |
give air presets a pressure target var. allows things to have non-standard pressures without affecting other systems.
in my mind, the pressure danger level should be just before the pressure is actually dangerous. |
Will you raise these changes or do you want me or someone else to make them |
looking at the jungle atmos numbers now, the ambient pressure is 1.2 atm (121.59 kPa), which is also (seems to be) exactly the max2 number. if you just increase it by .01 kpa (or even atm) or something, it'd solve both issues pretty easily |
Sorry if this comes across as turboretarded but is there a reason jungle is at 121 kPa rather than 101 kPa If anything this suggests the need for map-specific thresholds which goes far beyond the scope of this hotfix |
because that's as high as i could get it before zas started blowing things around. but if it's more of a "why is it not 1atm in the first place", then it's because it's a non-earth planet. why should it have 1 atmosphere of pressure? it's a creative choice to make the planet stand out more. similar reasoning to why the atmosphere has a small amount of CO2 in it. it's for flavor, as well as the gameplay implications that it allows. |
So either you need map-specific thresholds or one-size-fits-all thresholds, of which you seem to be pursuing the latter I will leave this to you and collabs to figure out |
should just merge this for now to go back to standard, |
Revert atmos |
This is the correct approach. I feared this happening on the initial Jungle PR and was told something like this wouldn't happen but it did. |
What this does
Fixes #38429
Reverts changes to atmos presets made as part of Junglestation. I believe the original intent behind these changes was to make Junglestation atmos not have under/overpressurisation issues, however the way that batch presets are applied via the CAC (target pressure is average of min1 and max1) results in target pressures for human presets (and others) getting set to an incorrect value. This leads to vents pushing more air into rooms than they should have and scrubbers quickly removing the un-needed excess.
How it was tested
Before change:


Batch Apply:
After change:


Batch Apply:
Changelog
🆑