Don't export `scitype` method.
Nadie ha tomado este issue todavía.
- #135 de @OkonSamuel — cerrado sin fusionar
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 42/100
Línea de trabajo
Comienza leyendo las definiciones y exportaciones de scitype en MLJModelInterface y, después, inspecciona el workaround en MLJBase/src/MLJBase.jl y el método correspondiente en MLJBase/src/Interface/data_utils.jl. Confirma que scitype ya no se exporta desde MLJModelInterface, que el uso cualificado sigue estando disponible y que el conflicto de importación del método se ha resuelto.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Due to a recent change in ScientificTypesBase,jl from version 2.0 to 3.0, the ScientifictypesBase.scitype method is now a two argument method. Packages depending on MLJModelInterface.jl previously depended on the previous one argument version defined in version 2.0 series of ScientifictypesBase package.
A workaround is needed to avoid making MLJModelInterface breaking for its's users. This involves
- Defining a one argument
scitypemethod in MLJModelInterface. Something in the lines of
scitype(X) = scitype(get_interface_mode(), X)
scitype(::LightInterface, X) = throw(ArgumentError("FullInterface is needed..."))
The corresponding FullInterface method would be implemented at MLJBase/src/Interface/data_utils.jl.
- exporting the
scitypemethod atMLJModelInterface(Needed as users expect that thescitypemethod is exported. see #129 )
But implementing the second step also leads to method import clashes with ScientificTypes.scitype at MLJBase. For now a hack at MLJBase/src/MLJBase.jl fixes this issue, but going forward the scitype method shouldn't be exported. Hence users that want to use the scitype method would have to qualify it with the package name e.g. MLJModelinterface.scitype(X).
- Lenguaje dominante
- Julia
- Estrellas
- 39
- Forks
- 8
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de JuliaAI/MLJModelInterface.jl
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
JuliaAI/MLJModelInterface.jl#215 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
JuliaAI/MLJModelInterface.jl#212 · 26 comentarios ·
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
JuliaAI/MLJModelInterface.jl#209 · 1 reacción ·
-
human_name vs Base.nameofAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
JuliaAI/MLJModelInterface.jl#208 · 2 comentarios ·
-
next major release
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
JuliaAI/MLJModelInterface.jl#189 ·
Todos los issues de JuliaAI/MLJModelInterface.jl
Issues similares
-
bignums
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
JuliaNLSolvers/NLsolve.jl#319 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
gridap/GridapPETSc.jl#129 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100