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

Movement update v2 #20548

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

FluffyGhoster
Copy link
Contributor

Refactored (second passage) how movement works, now it's mostly in line with TG handling and avoids calling 3 gazillion Cross() Uncross() etc. on every atom in a turf.
Fixed EMP protection from species not actually protecting (this includes the surge prevention for IPCs).
Fixed EMP 3D calculation runtiming because I forgot to make the value absolute and it was doing the square root of a negative number.
It's now possible to queue the round to start with the Start Round verb even while the system is initializing, for an even faster pain train to enter the round and test things.

@FluffyGhoster FluffyGhoster added ⚠ High Risk Reasonable chance of causing many or high-severity bugs. Test Merge Candidate The PR is eligible for test merging. labels Mar 3, 2025
@FluffyGhoster
Copy link
Contributor Author

!review

Copy link
Contributor

@alsoandanswer alsoandanswer left a comment

Choose a reason for hiding this comment

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

Hail the purity of the /tg/ code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠ High Risk Reasonable chance of causing many or high-severity bugs. Review Required Test Merge Candidate The PR is eligible for test merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants