Repository navigation
[ENH] Added ReprMixin to share __repr__ formatting - #1595
Merged
Merged
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1595 +/- ##
==========================================
+ Coverage 52.04% 52.71% +0.67%
==========================================
Files 36 36
Lines 4333 4350 +17
==========================================
+ Hits 2255 2293 +38
+ Misses 2078 2057 -21 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Contributor
|
@fkiraly the CI failures are coming from the production server - specifically database connection - probably because of overloading. Should we do something about these in short-term or just leave it for long-term test refactor? |
Contributor
|
There is one unrelated sporadic failure coming from the production server, due to high server load, which we are not skipping and should be fixed if rerun. |
PGijsbers
added a commit
that referenced
this pull request
Oct 15, 2026
…_repr__`, instead of copy-paste (#1567) <!-- Thanks for contributing a pull request to the OpenML python connector! Please ensure you have taken a look at the contribution guidelines: https://github.com/openml/openml-python/blob/main/CONTRIBUTING.md#Contributing-Pull-Requests Please make sure that: * the title of the pull request is descriptive * this pull requests is against the `develop` branch * for any new functionality, consider adding a relevant example * add unit tests for new functionalities * collect files uploaded to test server using _mark_entity_for_removal() * add the BSD 3-Clause license to any new file created --> #### Metadata * Reference Issue: Fixes: #1563 * New Tests Added: No * Documentation Updated: No * Change Log Entry: Inherit `ReprMixin` for common `__repr__` formatting across all `OpenML` classes * Dependent on #1595 --------- Co-authored-by: Franz Király <[email protected]> Co-authored-by: Eman Abdelhaleem <[email protected]> Co-authored-by: Rohan Sen <[email protected]> Co-authored-by: DDiyash <[email protected]> Co-authored-by: Armaghan Shakir <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Pieter Gijsbers <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Metadata
ReprMixinto share__repr__formatting across OpenML classes #1591ReprMixininutilsDetails
Had a question, Can I make the classes use the
mixinin this pr or should I open a new one?