-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
New release coordination #148
Comments
Cool! I want to investigate the NaN/infinity issue for a few more hours, some time in the next couple days. I agree there should be a new release soon :) |
The benchmarks section in the README is hopelessly out of date by now. We should either remove it or put new benchmarks there. However, do the particular numbers really matter? Perhaps we can provide an indication of roughly how much benefit each feature has and leave it at that. If people are interested at how fast the benchmarks are on their machines, they can easily run it. |
IMO the raw numbers don't matter. I'm in favor of removing them, but keeping the qualitative documentation about the relative speeds of different features. I'm also thinking about adding a phase diagram (per traversal type) to the README:
|
Sounds like a good idea. A phase diagram might be overkill. How about a bunch of easier to read triangles vs time diagrams? You can also be fancy if you want, though. |
I'd like to make a new release since there's been so much stuff in the mean time. Is there anything else you want to put into the next release @finnbear? Feel free to edit this and add a checklist with items.
The text was updated successfully, but these errors were encountered: