-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Recording performance improvements/regressions somewhere? #4724
Comments
The The |
@JeffBezanson probably a typo on my part somewhere... |
Somebody should watch out for regressions and report them here as you have done. Is there any way to annotate the data? |
We could come up with some method of doing so. Codespeed as it is attempts to do some basic statistical analysis on the data to identify "trends" or sharp changes based on statistics of the last 5 or so builds. We can fiddle around with what Codespeed does; that's the beauty of open source. :) |
So I've been looking at https://julia-codespeed.csail.mit.edu/ (I find it fascinating!) and there are some interesting things going on there, e.g.
fasta
test got a lot faster on Oct 18 (https://julia-codespeed.csail.mit.edu/timeline/#/?base=none&ben=fasta&env=1&revs=1000&equid=off&exe=1)go_benchmark
test got a fair bit faster on Oct 31 (https://julia-codespeed.csail.mit.edu/timeline/#/?base=none&ben=go_benchmark&env=1&revs=1000&equid=off&exe=1)pidigits
got a fair bit slower on Oct 16 (https://julia-codespeed.csail.mit.edu/timeline/#/?base=none&ben=pidigits&env=1&revs=1000&equid=off&exe=1)splitline
got slower on Oct 29 (https://julia-codespeed.csail.mit.edu/timeline/#/?base=none&ben=splitline&env=1&revs=1000&equid=off&exe=1)Should we be "annotating" these changes to explain what happened as part of long-term performance tracking? E.g. the
splitline
change seems to be a change in the test itself, but the commit that corresponds with the changes inpidigits
seems to be related to "LUFACT fixes for sparse complex matrices" (cdcc91d), which is odd.The text was updated successfully, but these errors were encountered: