Skip to content

Radiation Insulation Refactor Reopen#607

Merged
Ossa88 merged 15 commits into
PentestSS13:masterfrom
FslashN:NEW-RAD-REFACTOR-AAAAAA
May 23, 2026
Merged

Radiation Insulation Refactor Reopen#607
Ossa88 merged 15 commits into
PentestSS13:masterfrom
FslashN:NEW-RAD-REFACTOR-AAAAAA

Conversation

@FslashN
Copy link
Copy Markdown
Contributor

@FslashN FslashN commented Feb 28, 2026

About The Pull Request

Branched copy of: (#606)

Why It's Good For The Game

As the previous commit but all the modularization issues are now ironed out.

Changelog

🆑
refactor: Refactored radiation spread code as well as insulation.

FslashN and others added 5 commits December 11, 2025 18:25
Makes some adjustment to the way radiation waves spread through insulation. Previously as a radiation wave expanded, insulation (like walls, for example), did very little to stem the overall strength of the wave. With the change:

Insulation blockers have a greater impact on the overall insulation effect, reducing the radiation wave intensity. Greater insulation values also have more impact than before.
Only the best insulation is taken per tile, so it is not possible to stack insulation values. This change is done for sanity and to reduce computation.
Insulating blockers were expanded and adjusted. Blast doors now have great insulation, for example. Additionally, if you open a door, it will have an impact on the overall radiation wave as the door no longer will act as insulation. Previously, doors and other obstacles that could open and closed always counted as an insulation blocker.
Fixing merge problems with TMs.
@FslashN FslashN changed the title New rad refactor aaaaaa Radiation Insulation Refactor Reopen Feb 28, 2026
Modularizes stuff, thanks to @Ossa88 for a lot of the original modularization work.
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

FslashN added 6 commits March 11, 2026 20:17
Looks like some lines were removed.
I forgot to actually remove the changes to some individual files.
Reverting a few more lines I missed.
Forgot to remove two lines. Oops.
@Ossa88
Copy link
Copy Markdown
Collaborator

Ossa88 commented Mar 12, 2026

Love how there was a merge conflict that wasnt a real merge conflict and github only told me well after i merged small things

@FslashN
Copy link
Copy Markdown
Contributor Author

FslashN commented Mar 12, 2026

It's failing for some strange reason. Not really sure.

@Ossa88
Copy link
Copy Markdown
Collaborator

Ossa88 commented Mar 12, 2026

It's failing for some strange reason. Not really sure.

Thats the DMI 5 checker from paradise... it might be down atm

Just allow the other tests as those are generally more important.

@github-actions
Copy link
Copy Markdown
Contributor

This PR has been inactive for a month. This means it is at risk of being auto closed in a week, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions Bot added the Stale label May 21, 2026
@Ossa88 Ossa88 enabled auto-merge May 23, 2026 08:35
@Ossa88 Ossa88 added this pull request to the merge queue May 23, 2026
Merged via the queue into PentestSS13:master with commit 705cbd6 May 23, 2026
23 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants