-
Notifications
You must be signed in to change notification settings - Fork 22
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
Agent Sorting #723
Agent Sorting #723
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.
Suggest that each of the unit tests gives a descriptive comment to what the test is doing.
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.
I might be on the wrong page, but I think some of this needs discussion at the next meeting.
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.
I'm happy, Paul had a few comments at the meeting?
This PR includes support for sorting agents which use spatial messaging by their position to accelerate message read times.
Initial results show that the ideal sorting frequency depends on the model, both boids and circles have achieved a runtime improvement in the range of 20%-40% depending on sort frequency.