-
Notifications
You must be signed in to change notification settings - Fork 111
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
smart analysis improvements #921
Comments
If we use random sampling for both the initial estimation and the later analysis, then the resulting CI should be really close to the specified desired value |
I do not think disabling by default will be good. I would say most regular users want okay-ish results in a short amount of time. The other features I do like. Also perhaps it will be useful to add an option to limit the data analyzed yourself. for example, if the script says that you need 230,000 points but you want to push it a bit more to be able to specify (via a flag) to use 250,000 points. What do you think about that? |
Notes:
|
Feature request
Is your feature request related to a problem? Please describe
The feature works as described, in that when one uses defaults, this is equal to asking for one nanosecond CIs, and the resulting CIs are ~1ns wide.
The feature doesn't work as expected, in that when one might approach a value close to a nanosecond, yet larger than a nanosecond, keep collecting data, and never have their CIs improve because the feature rejects extra data.
Describe the solution you'd like
Describe alternatives you've considered
Additional context
The text was updated successfully, but these errors were encountered: