-
Notifications
You must be signed in to change notification settings - Fork 24
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
Merge dev in main β¨ #24
Conversation
o-laurent
commented
May 26, 2023
•
edited
Loading
edited
- Update packages β‘
- Remove useless functional transforms π₯
- Add a link to Packed-Ensembles medium post π
- Add ImageNet variations β¨
- Don't compute coverage on datasets and load_pretrained methods
- Merge baselines simplifications PR π¨
- Add tests (missing for transforms & pixmix)
- Add update package procedure
Increase legibility of the raw Readme
Slightly improve docstrings π
Add a dummy network Make the dummy dataset class more flexible
Add the pre-commit-hooks package β Solve the new hooks' issues π Create a contributing page π Add a reference to WR π
Add dummy baselines and datamodules β¨ Fix dummy models and datasets π
Temporary solution for the note issue on the installation page :docs:
Fix tests βοΈ
Rework baselines
The current code is acceptable to me for merging. |
Codecov ReportPatch coverage:
β Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. Additional details and impacted files@@ Coverage Diff @@
## main #24 +/- ##
==========================================
+ Coverage 86.75% 91.88% +5.12%
==========================================
Files 90 81 -9
Lines 3291 3449 +158
Branches 231 287 +56
==========================================
+ Hits 2855 3169 +314
+ Misses 351 190 -161
- Partials 85 90 +5
Flags with carried forward coverage won't be shown. Click here to find out more.
β View full report in Codecov by Sentry. |
Add tests for transforms β
Fix link π
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 believe the state of this pull request is fine enough to be merged.