-
Notifications
You must be signed in to change notification settings - Fork 60
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
FIX benchmark_utils pickling #713
Conversation
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.
I let you merge this time 😉
Let's just wait for confirmation that this fixes the issue of @antoinecollas :) |
It works! Thanks a lot Thomas 🙏🏻 |
@agramfort feel free to merge once it is green 😉 |
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #713 +/- ##
==========================================
- Coverage 49.51% 49.42% -0.10%
==========================================
Files 45 45
Lines 3009 3019 +10
Branches 570 572 +2
==========================================
+ Hits 1490 1492 +2
- Misses 1376 1383 +7
- Partials 143 144 +1 |
thx @tomMoral ! |
* FIX benchmark_utils pickling * TST add some non-regression test * DOC add what's new entry+explain test
Follow up of #712 which was merged a bit quickly.
This should fix the issue in scikit-adaptation/skada-bench#60 with missing
benchmark_utils
module.