-
Notifications
You must be signed in to change notification settings - Fork 1
quadtree network topology #62
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
Conversation
karalekas
left a comment
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.
also I would really like to see that unit test or two
karalekas
left a comment
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.
also, as discussed offline, id be very interested to see a third network variant that includes both NN and quadtree links, and prioritizes NN links when using them would result in a better routing time
while ofc the quadtree routing time is asymptotically better for long-distance routing, we expect in practice to do a lot of NN routing and it seems non-ideal to double the NN routing time in the quadtree network configuration
|
added a minimal unit test; you'll need to pull @ecpeterson |
11bfc6a to
448d2e9
Compare
karalekas
left a comment
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.
changes LGTM, thanks!!
TODO:
ceil.little demo: