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

Mirror 26441: Fix atmos NaN error #449

Conversation

SimpleStation14
Copy link
Member

Mirror of PR #26441: Fix atmos NaN error from space-wizards space-wizards/space-station-14

fdb4a61487db9fc67714c913832427063abdea42

PR opened by ElectroJr at 2024-03-26 04:27:08 UTC - merged at 2024-03-26 04:44:56 UTC


PR changed 16 files with 43 additions and 25 deletions.

The PR had the following labels:

  • Status: Needs Review

Original Body

  • Fixes a bug I introduced in #22521 that caused the temperature to be set to NaN. I don't know for sure, but I assume this is causing the current atmos issues
  • Fixes FixVacuum not working after #22521
  • Removes some redundant yaml that was setting FixVacuum to its default value

🆑

  • fix: Fixed an atmos bug, which was (probably) causing atmospherics on the station to behave incorrectly.

@SimpleStation14 SimpleStation14 added the Pull Request Mirror Mirrors a PR from another Repo. Automatically applied by mirror bot label Jun 11, 2024
@Pspritechologist Pspritechologist added Status: Derelict Has not been active for a while and removed Status: Derelict Has not been active for a while labels Jun 11, 2024
@VMSolidus VMSolidus marked this pull request as ready for review June 12, 2024 01:01
Copy link
Member

Choose a reason for hiding this comment

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

What is this doing here?? 🤨

Copy link
Member

Choose a reason for hiding this comment

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

I'm genuinely not sure why it has to diff the entire entity, but the only thing being removed is the Airtight fixVacuum argument.

Copy link
Member

Choose a reason for hiding this comment

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

Turning on 'hide whitespace' removes the odd changes, so just some funky whitespace stuff. Consider redoing the important changes so we don't have the ugly diff, merge otherwise.

Copy link
Member

@VMSolidus VMSolidus left a comment

Choose a reason for hiding this comment

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

I need this for the next set of Engine updates.

Copy link
Member

Choose a reason for hiding this comment

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

I'm genuinely not sure why it has to diff the entire entity, but the only thing being removed is the Airtight fixVacuum argument.

Copy link
Member

Choose a reason for hiding this comment

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

Turning on 'hide whitespace' removes the odd changes, so just some funky whitespace stuff. Consider redoing the important changes so we don't have the ugly diff, merge otherwise.

@VMSolidus VMSolidus merged commit 2138b8a into Simple-Station:master Jun 13, 2024
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Pull Request Mirror Mirrors a PR from another Repo. Automatically applied by mirror bot
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants