-
Notifications
You must be signed in to change notification settings - Fork 87
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
Reproducibility fixes #1278
base: master
Are you sure you want to change the base?
Reproducibility fixes #1278
Conversation
👋 Hi, I'm @docu-mentor, an LLM-powered GitHub app Simply create a new comment in this PR that says: @docu-mentor run and I will start my analysis. I only look at what you changed @docu-mentor run doc/ README.md In this example, I'll have a look at all files contained in the "doc/" |
All PEP8 errors has been fixed, thanks ❤️ Comment last updated at |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1278 +/- ##
==========================================
+ Coverage 79.92% 79.95% +0.02%
==========================================
Files 146 146
Lines 10049 10054 +5
==========================================
+ Hits 8032 8039 +7
+ Misses 2017 2015 -2 ☔ View full report in Codecov by Sentry. |
Hello @nicl-nno! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2024-06-27 10:23:53 UTC |
OpTuna-based tuning is still non-reproducible in multi-threaded mode (by design)