[MNT] merge `openml-sklearn` extension back into `openml`
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 25/100
- Type d'issue
- Refactorisation
- Clarté
- À clarifier
- Activité
- À l'abandon
- Stack technique
- python
- Domaine
- machine-learning
Piste de recherche
Start by comparing the openml-sklearn extension with the openml-python package and reviewing their release and maintenance workflows. Evaluate the proposed merge while preserving scikit-learn dependency isolation through soft dependencies or class/function-level patterns. Done means the extension is merged into openml-python without making scikit-learn a hard dependency for unrelated functionality.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
openml-sklearn is in unmaintained - likely due to splitting packages always dramatically increases maintenance surface and complexity. Once release and maintenance workflows are "forgotten", it leads to abandoment.
Of course there are reasonable rationales behind splitting packages, such as dependency isolation.
My recommendation would be to merge openml-sklearn back into openml, in order to minimize maintenance surface - but also maintain dependency isolation.
My concrete recommendations would be:
- plain merge of
openml-sklearnintoopenml-python - use
scikit-base_check_soft_dependencies, or "move into class/function" patterns, to isolate thescikit-learndependency in the extension and the tests of the extension - move
scikit-learncompletely to a soft dependency set, e.g.,openml-integrations
FYI @joaquinvanschoren, @PGijsbers - what do you think? I think we need to avoid abandonment due to too complex maintenance. Dependency isolation can be maintained by using proper patterns within a single package (not just repository).
- Langage dominant
- Python
- Étoiles
- 361
- Forks
- 296
- Merge moyen
- 2 j 20 h
- PR mergées (30 j)
- 2
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de openml/openml-python
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
openml/openml-python#1750 · 3 commentaires · 1 personne assignée ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
openml/openml-python#1749 ·
-
Documentation Good First Issue
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
openml/openml-python#1708 · 7 commentaires ·
-
Good First Issue module:Run testing
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
openml/openml-python#1646 · 7 commentaires · 2 réactions ·
-
Good First Issue module:Data testing
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
openml/openml-python#1644 · 4 commentaires ·
Toutes les issues de openml/openml-python
Issues similaires
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 75/100
-
hcocena Ouvertepolicies-accepted pre-review precheck-passed
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
Bioconductor/BiocContributions#214 · 5 commentaires ·
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 92/100
TencentCloud/Octop#1169 · 1 commentaire ·
-
[开源推荐] 在老板拷问你之前,先让 AI 灵魂拷问你 Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
521xueweihan/HelloGitHub#3778 ·
-
The version checker's trailing attribute region has no control for a less-than inside a quoted value Ouvertearea: dashboard area: tests bug perceived difficulty: 2 python
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Nitjsefnie-Harness-Commons/daedalus#1105 · 1 commentaire ·