discordia-space / CEV-Eris

Death is our destination
https://discord.gg/QfEg7K7
Other
152 stars 496 forks source link

Mapped Central Fuel Chamber, plus various atmos related troubleshooting and map fixes #8470

Closed Mycah142 closed 6 months ago

Mycah142 commented 7 months ago

About The Pull Request

What started as just making a central fuel tank turned into "What do the numbers MEAN?!", and realized Atmospherics depowering was due to map var edited devices not being done well. ShareX_NgFiIeezpk

Here's a step by step as to what I changed: (I tried to do spoilers, but then the images didn't embed properly. I don't know.)

Why It's Good For The Game

Atmospherics shouldn't depower within 5 seconds (it's five minutes now ha), and thruster setup should be easier to understand. With a central tank that doesn't need individual thruster set up, it's also faster to do. Select your mix, open the valves, and then open the manual valve outside the window. Previously, when the chamber appeared empty, players thought that fuel was empty, but it was actually entirely in the piping.

Also bugfixes good. The bugfixes involved alot of map editing, so they're part of this PR as they were related to making the Thruster Chambers work as expected.

Testing

Trust me bro.

Please see the summaries above. I spent alot of time learning how these items work to try and make the best solution.

Changelog

:cl: add: Central Thruster Fuel Chamber, and related valves to make it work. del: Pipe confusion in individual thruster rooms, separating Central Chamber from Emergency Chambers. del: Supermatter related tips from job text. tweak: Vent processing to introduce a hibernation, and temperature trigger. tweak: Fuel chamber venting no longer a passive vent, now shutters. fix: Map Var Edited gas chamber vents to obey internal pressure, now only should be map edited for ID Tags. fix: Mixing chamber gas sensors should now display all gas contents in Gas Composition. /:cl:

Mycah142 commented 7 months ago

Working to fix map now.

Mycah142 commented 7 months ago

didn't mean to push, god dammit.

Mycah142 commented 7 months ago

I'll have this one ready for next week, as the amount of mapping needs to be retested. If anything, I could remove the DMM and we can just go with the optimization.

Mycah142 commented 7 months ago

I have no idea why I had to "update from master" three times, with one of those commits containing zero changed files, and each time the map was perfect on my end, but whatever, you do you github desktop.

Good news, I learned how to force a new head commit today using the terminal.