operate_fallback!! shouldn't force the first argument to be copied
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- julia
- Área
- backend-api-design
Línea de trabajo
Comienza en src/interface.jl, en la redirección alrededor de las líneas 592-599, y sigue cómo operate_fallback!! delega en operate, especialmente a través de @rewrite. Compara el tratamiento del primer argumento con el de los demás argumentos y verifica que el resultado pueda depender únicamente de ese primer argumento o modificarlo. Comprueba el uso canónico en StarAlgebras.jl para el comportamiento descrito, sensible al rendimiento.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
In this redirection:
https://github.com/jump-dev/MutableArithmetics.jl/blob/7344c9468072bd663feec7e4cdc1a0e98e34ea19/src/interface.jl#L592-L599
We redirect to operate because for instance, when it's used in the @rewrite macro, we want to modify the result so it cannot depend on the the arguments.
However, since the first argument was given to operate!!, it can actually be modified! So in this case, we want a result that does not depend on any argument except the first one.
This can causes performance issues in https://github.com/JuliaAlgebra/StarAlgebras.jl/pull/21/files#diff-3f191d1ba6c7bc538c1bf193064434a4984a0457baac3da62c9e6dede5d566aa for canonical as reported by @kalmarek
- Lenguaje dominante
- TeX
- Estrellas
- 55
- Forks
- 15
- Merge medio
- 17 h 21 min
- PR fusionados (30 d)
- 8
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 jump-dev/MutableArithmetics.jl
-
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
jump-dev/MutableArithmetics.jl#353 · 4 comentarios ·
-
Documentation needs to be addedAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
jump-dev/MutableArithmetics.jl#293 · 8 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
jump-dev/MutableArithmetics.jl#253 · 7 comentarios ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 58/100
Todos los issues de jump-dev/MutableArithmetics.jl
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Azure-Samples/azure-search-openai-demo#3249 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
Los mantenedores suelen responder en 1 día
-
Type: Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
espresso3389/pdfrx#776 ·
Los mantenedores suelen responder en 1 día