ArrayRef instead of ArrayBase in function interfaces
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 58/100
Línea de trabajo
Comienza con las implementaciones y la documentación de PyArray::borrow_from_array y PyArray::from_array; después, inspecciona impl ToPyArray for ArrayRef y la declaración de la dependencia ndarray. Comprueba todos los sitios de llamada afectados y las comprobaciones existentes del proyecto; el trabajo estará terminado cuando las interfaces propuestas y el requisito de dependencia se hayan actualizado de forma coherente y el proyecto siga compilando.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hi,
I frequently find myself wanting a read-only array in Python backed by a Rust-owned buffer (usually to print it in an interactive session).
For this I use PyArray::borrow_from_array which takes a &ArrayBase as parameter.
I would like this function to take a &ArrayRef instead to be able to use it with a greater variety of arguments.
While this is a breaking change, I reckon the necessary code changes at call sites would be small, since ArrayBase already implements Deref<Target=ArrayRef>.
In my quick search I only found a couple of places, where this proposed change could be applied
PyArray::borrow_from_array,PyArray::from_array,impl ToPyArray for ArrayRef(? not sure about this one).
Another consequence would be to bump the minimum version of the ndarray dependency to 0.17.1.
Kind regards!
- Lenguaje dominante
- Rust
- Estrellas
- 1.4k
- Forks
- 141
- Merge medio
- 15 min
- PR fusionados (30 d)
- 1
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 PyO3/rust-numpy
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
PyO3/rust-numpy#565 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 56/100
PyO3/rust-numpy#557 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
PyO3/rust-numpy#547 · 2 comentarios ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 20/100
PyO3/rust-numpy#535 ·
-
panic in as_array_mutAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
PyO3/rust-numpy#527 · 1 comentario ·
Todos los issues de PyO3/rust-numpy
Issues similares
-
Change output crossing a compactsize boundary leaves the fee slightly below the requested feerateAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
bitcoindevkit/bdk_wallet#578 ·
Los mantenedores suelen responder en 8 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
521xueweihan/HelloGitHub#3832 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
canonical/opentelemetry-collector-operator#409 ·
Los mantenedores suelen responder en 1 día