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
Various fixes around timing tests #795
base: master
Are you sure you want to change the base?
Conversation
This pull request introduces 2 alerts when merging 3f67967 into c56a143 - view on LGTM.com new alerts:
|
a4f6937
to
e251ac8
Compare
This pull request introduces 2 alerts when merging e251ac8 into ad0ec24 - view on LGTM.com new alerts:
|
e711756
to
dcbaa84
Compare
docs/source/timing-analysis.rst
Outdated
dnf install -y python3 python3-devel tcpdump gmp-devel swig mpfr-devel \ | ||
libmpc openssl-devel make gcc gcc-c++ git libmpc-devel python3-six \ | ||
meson openblas-devel gcc-gfortran lapack-devel zlib-devel \ | ||
libtiff-devel libjpeg-devel openjpeg2-devel zlib-devel freetype-devel \ |
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.
zlib-devel is duplicated
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.
actually, given the complexity of getting them all installed, I'll probably provide both commands, as it affects only people running tests on platforms other than x86_64 and aarch64
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.
fixed
2019f53
to
4ac68d4
Compare
2ecf808
to
a01bd27
Compare
tlsfuzzer/analysis.py
Outdated
diff_stats["SD"] = np.std(diff) | ||
quantiles = np.quantile(diff, [0.25, 0.5, 0.75]) | ||
diff_stats["median"] = quantiles[1] | ||
diff_stats["IQR"] = quantiles[2] - quantiles[1] |
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.
this is not Q3-Q1...
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.
fixed
@t184256 why did you push to this branch? |
By mistake. |
ce91c3d
to
26e6932
Compare
b370fdc
to
9454b60
Compare
c7a7c38
to
c74809d
Compare
use to create a list of IDs from the legend file to allow analysis of raw_times_detail.csv based on probe ID include also the tuple ID for pairwise testing
Description
Make the timing tests more useful/usable/reliable
Motivation and Context
Better handling of timing tests
Obsoletes #728
Checklist
tlslite-ng.json
andtlslite-ng-random-subset.json
This change is