Inadequate support of `promote_rule` for non-concrete types
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 35/100
Línea de trabajo
Reproduce the listed promote_rule cases in the Julia REPL and read the handling in ColorTypes.jl. Compare the behavior with issue #342, then establish expected results for the examples, including avoiding StackOverflowError. Done means the non-concrete promotion cases behave consistently without the reported errors.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
For better or worse, PR #207 is the most significant change in FPN v0.9.
However the current promote_rule for non-concrete types is somewhat imperfect.
In ColorTypes.jl, I am keenly aware that the handling of non-concrete types can lead to chaos.
Here, I’d like to think about ways to make things a little better, as long as they don’t go against common sense.
julia> promote_rule(Normed{UInt8, 8}, Fixed{Int8, 7})
Float32
julia> promote_rule(Normed{UInt8}, Fixed{Int8})
Union{} # -> Float32
julia> promote_rule(Normed{UInt8}, Normed{UInt8})
Union{} # -> Normed{UInt8}
julia> promote_rule(Normed{UInt8}, Normed{UInt16})
Union{} # -> Normed{UInt16}
julia> promote_rule(Normed{UInt8}, Normed)
Union{} # -> Normed
julia> promote_rule(Normed{UInt8}, Normed{Union{UInt8, UInt16}})
Union{} # -> Normed (Any desire to make this more strict is a path to despair.)
julia> promote_rule(Normed{UInt8}, FixedPoint)
Union{} # -> FixedPoint
julia> promote_rule(Normed{UInt8}, Float64)
Float64
julia> promote_rule(Normed, Float64)
ERROR: StackOverflowError # -> BigFloat
julia> promote_rule(Normed{UInt8}, Int8)
Float32
julia> promote_rule(Normed, Int8)
Normed # -> BigFloat
Some of the unnatural promotions or stack overflow errors are caused by issue #342. In other words, fixing issue #342 will resolve some of these issues.
- Lenguaje dominante
- Julia
- Estrellas
- 95
- Forks
- 38
- Merge medio
- 16 h 31 min
- PR fusionados (30 d)
- 9
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
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 JuliaMath/FixedPointNumbers.jl
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 62/100
JuliaMath/FixedPointNumbers.jl#342 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
JuliaMath/FixedPointNumbers.jl#337 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 42/100
JuliaMath/FixedPointNumbers.jl#331 · 7 comentarios ·
Los mantenedores suelen responder en 1 día
-
needs comments
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
JuliaMath/FixedPointNumbers.jl#318 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
add-on enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
JuliaMath/FixedPointNumbers.jl#311 · 4 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de JuliaMath/FixedPointNumbers.jl
Issues similares
-
add-on doc enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
JuliaGraphics/ColorTypes.jl#344 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
R dockerfile no longer buildsAbiertoR
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
eWaterCycle/remotebmi#58 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
`enzymexla.linalg.lu` lowering fails for a tall matrix: the permutation is built with the pivot typeAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
EnzymeAD/Enzyme-JAX#3286 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 90/100
SciML/LinearSolve.jl#1359 · 1 comentario ·
Los mantenedores suelen responder en 1 día