| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
…ayL_A.rst Co-authored-by: Wulian233 <1055917385@qq.com>
|
Thanks for doing this. I did it for hmac locally since I'm implementing the HACL* hmac nut it's good to see it done here. |
Sorry, something went wrong.
Co-authored-by: Vinay Sajip <vinay_sajip@yahoo.co.uk>
There was a problem hiding this comment.
Please revert the whitespaces and unrelated new lines changes (unless the codeowners tell you that you can keep the changes though)
Sorry, something went wrong.
|
A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated. Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request. |
Sorry, something went wrong.
|
We should verify that each change has non-noise benefits rather than changing everything at once in an omnibus PR. @donbarbos also please present your motivating statistics in a more readable manner, I expect to see a comparison of each module against baseline, rather than four large log outputs. A |
Sorry, something went wrong.
|
I ran a test for importlib.resources, and this change if anything makes things worse: cpython issue-118761 🐚 hyperfine --warmup 11 "./python.exe -c 'import importlib.resources'" Benchmark 1: ./python.exe -c 'import importlib.resources' Time (mean ± σ): 26.6 ms ± 0.5 ms [User: 20.9 ms, System: 3.8 ms] Range (min … max): 25.4 ms … 28.7 ms 101 runs cpython issue-118761 [1] 🐚 git checkout @$(git merge-base HEAD main) HEAD is now at 0fef47e5bbd gh-55454: Add IMAP4 IDLE support to imaplib (#122542) cpython 0fef47e5bbd 🐚 hyperfine --warmup 11 "./python.exe -c 'import importlib.resources'" Benchmark 1: ./python.exe -c 'import importlib.resources' Time (mean ± σ): 25.9 ms ± 0.6 ms [User: 21.4 ms, System: 3.8 ms] Range (min … max): 24.8 ms … 27.6 ms 98 runs Sounds like we should revert those changes from this PR. |
Sorry, something went wrong.
|
I'll close this PR as the contributor has opened some smaller individual changes. A |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
My hyperfine benchmarks:
First experiment in Main branch
Second experiment in Main branch
First experiment in PR branch
Second experiment in PR branch