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 #8584: Duck spawn search now uses entire map #9743
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would prefer if we are touching this to refactor to use CoordsXY where we can.
If this gets rebased and changed to use coords which might be easier now after @duncanspumpkin's refactor, we can merge. |
Sorry got a little distracted by other things, will do this today. |
Applied the requested changes and cleaned up the code a little bit. Also I bumped up the network version since scenario_rand is now called twice. |
Could you rebase this and rebump the network version? I think it's time to get this merged, and Duncan's requested changes have been made anyway. |
e7c7dce
to
03ecfb7
Compare
Rebased and incremented the network version |
Requested changes have been made
This one should be without crashes. I left a few asserts in there to be sure and they have no performance impact.