Skip to content

Make gases use alist#5248

Open
out-of-phaze wants to merge 1 commit into
NebulaSS13:devfrom
out-of-phaze:experiment/gas-alist
Open

Make gases use alist#5248
out-of-phaze wants to merge 1 commit into
NebulaSS13:devfrom
out-of-phaze:experiment/gas-alist

Conversation

@out-of-phaze
Copy link
Copy Markdown
Member

Description of changes

gases use alist now. also includes some attempts at simplifying the math. this is probably extremely buggy

Why and what will this PR improve

in theory this will be a good bit faster.

Authorship

me

@out-of-phaze out-of-phaze added help wanted Extra attention is needed work in progress This PR is under development and shouldn't be merged. labels Jan 1, 2026
@out-of-phaze out-of-phaze added ready for review This PR is ready for review and merge. and removed work in progress This PR is under development and shouldn't be merged. labels Jan 23, 2026
@out-of-phaze out-of-phaze marked this pull request as ready for review January 23, 2026 00:07
@out-of-phaze out-of-phaze removed the help wanted Extra attention is needed label Apr 11, 2026
@out-of-phaze
Copy link
Copy Markdown
Member Author

cc @noelle-lavenza, i think some issues with my math were found on nuclear phase that need to be backported?

Comment thread code/modules/xgm/xgm_gas_mixture.dm
Comment thread code/modules/power/singularity/collector.dm
Comment thread code/modules/materials/_materials.dm
@MistakeNot4892 MistakeNot4892 added awaiting author This PR is awaiting action from the author before it can be merged. has conflicts This PR needs updating and conflict resolution before it can be merged. and removed ready for review This PR is ready for review and merge. labels May 8, 2026
@out-of-phaze out-of-phaze force-pushed the experiment/gas-alist branch from e6e95f4 to 7cdb881 Compare May 9, 2026 14:38
@out-of-phaze out-of-phaze added ready for review This PR is ready for review and merge. and removed awaiting author This PR is awaiting action from the author before it can be merged. has conflicts This PR needs updating and conflict resolution before it can be merged. labels May 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review This PR is ready for review and merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants