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

Fix attrition freezing and killing servers sometimes #642

Merged

Conversation

ASpoonPlaysGames
Copy link
Contributor

@ASpoonPlaysGames ASpoonPlaysGames commented May 11, 2023

More specifically, it happened when all enemy NPCs were dead and an NPC squad tried to find somewhere to attack. They wouldn't find any valid places to attack, so naturally you just continue in the loop and try again next time... except you forgot to wait any amount of time before trying again, so it's just going to try again in that same frame, forever.

Regression caused by #626 and rly should've been caught in testing or code review tbh

Closes R2Northstar/Northstar#483

@ASpoonPlaysGames
Copy link
Contributor Author

Add the "needs testing" and "needs code review" tags, I DARE YOU

Copy link
Member

@pg9182 pg9182 left a comment

Choose a reason for hiding this comment

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

Makes sense, and I couldn't reproduce this issue while solo grunting (hehe, grunt-hunting) after applying this patch.

We should write about this in the wiki and the troubleshooting docs.

@pg9182
Copy link
Member

pg9182 commented May 11, 2023

Note: The buggy PR is in NS 1.13.2+.

Copy link
Member

@GeckoEidechse GeckoEidechse left a comment

Choose a reason for hiding this comment

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

PR was tested by two different hosters and no crashes in regards to the original issue were experienced.

@GeckoEidechse GeckoEidechse merged commit d205d44 into R2Northstar:main May 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Attrition server crashing since v1.13.0
3 participants