-
Notifications
You must be signed in to change notification settings - Fork 6
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
HGEdge memory management / speed #453
Conversation
* HGEdge memory management * area graph speedup * nix route_names_and_systems * Args::numthreads cleanup interactive rebase d04ac12af81e27d8e9886345b57ee5533cb9722e
|
area graph edge-matching speedcloses yakra#176 A decent improvement for area graph speed -- although area graphs are already so fast it's barely any difference in the big picture. Python:
|
nix route_names_and_systemscloses yakra#175
|
Close yakra#182Minor code organization. |
Bit shifts in
|
box | before | after |
---|---|---|
BT | 103.150 | 99.930 |
lab1 | 65.470 | 61.330 |
lab1.5 | 52.830 | 50.640 |
lab2 | 77.810 | 73.480 |
lab3 | 103.730 | 97.410 |
lab4 | 78.400 | 74.510 |
BSDlab | 131.61 | 127.06 |
area graphs:
box | before | after |
---|---|---|
BT | 1.910 | 1.720 |
lab1 | 1.130 | 0.970 |
lab1.5 | 0.900 | 0.820 |
lab2 | 1.350 | 1.220 |
lab3 | 1.810 | 1.570 |
lab4 | 1.380 | 1.250 |
BSDlab | 2.27 | 2.02 |
Pulling this onto noreaster to use starting with tonight's update. Thanks! |
closes yakra#175
closes yakra#176
closes yakra#182
Yes, there's still room to squeeze some more efficiency out of writing subgraphs.
Good RAM savings. Speedwise, we have several nickel-and-dime improvements that add up to like a quarter. ;P
Code reorganization better sets us up for potential future enhancements.
Most everything squashed down into one commit for cleaner commit history, as successive commits edited and re-edited the same code lines.
All affected tasks together:
Broken down into the individual subtasks:
More detailed comments about the individual changes will be in the posts below.