Stack overflow when broadcasting results in StaticInt eltype
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- julia
- Área
- performance
Línea de trabajo
Empieza reproduciendo ambos ejemplos en Julia y, después, inspecciona src/static.jl:108 y el archivo base/range.jl de Julia alrededor de Base._range y de las líneas 107-118. Sigue cómo se construyen los rangos StaticInt y cómo el slicing llega a StepRange; se considera terminado cuando el broadcasting ya no produce overflow y la expresión de rango con paso mostrada ya no genera el MethodError indicado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Found this...
julia> import ArrayInterface: StaticInt
julia> r = StaticInt(1):4
StaticInt{1}():StaticInt{1}():4
julia> broadcast(-, r, StaticInt(1))
ERROR: StackOverflowError:
Stacktrace:
[1] UnitRange{StaticInt{0}}(::StaticInt{0}, ::StaticInt{3}) at /Users/zchristensen/.julia/packages/ArrayInterface/NbkVT/src/static.jl:108 (repeats 79984 times)
...and traced the issue back to Base._range assuming that UnitRange should be formed with an element type that is a StaticInt.
The easiest way to fix this would be our own Base._range(::StaticInt, ...), but I'm not sure if that's a good entry point since it's not public API.
Edit:
also found this
julia> (StaticInt(1):10)[1:2:6]
ERROR: MethodError: no method matching StepRange{StaticInt{1},Int64}(::StaticInt{1}, ::Int64, ::StaticInt{5})
Closest candidates are:
StepRange{StaticInt{1},Int64}(::T, ::S, ::T) where {T, S} at range.jl:204
Stacktrace:
[1] _rangestyle(::Base.Ordered, ::Base.ArithmeticWraps, ::StaticInt{1}, ::Int64, ::Int64) at ./range.jl:118
[2] _range at ./range.jl:116 [inlined]
[3] #range#43 at ./range.jl:91 [inlined]
[4] getindex(::ArrayInterface.OptionallyStaticUnitRange{StaticInt{1},Int64}, ::StepRange{Int64,Int64}) at ./range.jl:713
[5] top-level scope at REPL[26]:1
- Lenguaje dominante
- Julia
- Estrellas
- 14
- Forks
- 8
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Aún no hemos revisado los archivos de configuración de este proyecto. 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 JuliaArrays/StaticArrayInterface.jl
-
tagbot-manual
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
-
Storage OrderAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
JuliaArrays/StaticArrayInterface.jl#4 · 1 comentario ·
-
Size of StaticArray typeAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 32/100
JuliaArrays/StaticArrayInterface.jl#3 · 10 comentarios ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
Todos los issues de JuliaArrays/StaticArrayInterface.jl
Issues similares
-
`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 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
EnzymeAD/ReactantNitro.jl#13 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día