Hola Gabriel, intentaré esto durante el fin de semana.
Tener esto vinculado en algún lugar sería bueno, pero creo que es demasiado nicho para incluirlo en numpy. solo queremos las versiones nan*
de las funciones más utilizadas, y ptp
no es una de esas.
Al observar las funciones nan*
, parece que la mayoría de las funciones básicas tienen análogos nan. Estoy de acuerdo en que ptp
no es la función más utilizada, pero la uso bastante en problemas de procesamiento de señales, y parece que falta dadas las otras funciones en nanfunctions.py
. Terminé de implementar una versión básica, la instalaré en mi cuenta una vez que esté limpia y probada.
Si decidimos seguir adelante con este, acabo de abrir un PR con la implementación de la función y las pruebas correspondientes.
Hablamos de esto en la reunión de clasificación. La mayoría de las personas se alegraron de rechazar esta solicitud de función. @charris se abstuvo, @seberg dijo que podía imaginar un espacio de nombres separado.
La razón principal aquí es que no queremos más funciones nan en general; tenemos los más utilizados, y no se justifica llenar el espacio de nombres principal con más funciones nan (hay un costo significativo por agregar algo al espacio de nombres en este momento).
En particular para este: ptp
ya es un nicho, y podría decirse que fue un error agregarlo a numpy. Entonces nanptp
no es deseado.
Un paquete separado como bottleneck
tiene más sentido para más funciones nan.
Gracias por sugerir esto @ Gabriel-p, y por el PR @yliapis
Gracias por la explicación detallada @rgommers
Comentario más útil
Hola Gabriel, intentaré esto durante el fin de semana.