-
Notifications
You must be signed in to change notification settings - Fork 21
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
Implement confidence intervals #55
Implement confidence intervals #55
Conversation
4f18ba5
to
57b69c7
Compare
Code changes look good to me. But looks like |
Yeah, I noticed this at smaller packet sizes... |
Seems relevant - esnet/iperf#1078 also, w/ jumbo frames, and smaller message sizes (1024 or really anything < 4096) w/ iperf3 we see a significant difference between netperf and iperf3. |
Can you push another commit after running; |
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.
Core logic looks good to me except for the workflow check failures.
locally it failed to build for me, i had to run
|
57b69c7
to
542510a
Compare
Resolves: cloud-bulldozer#42 Signed-off-by: Sai Sindhur Malleni <[email protected]>
542510a
to
89f1cb3
Compare
@smalleni @jtaleric , some of the research on the iperf3 side about this in case you have interest: |
Resolves: #42
Confidence Interval improves with number of samples - example below