Oi Gabriel, vou tentar no fim de semana
Ter esse link em algum lugar seria bom, mas acho que é muito nicho para incluir no numpy. queremos apenas nan*
versões das funções mais comumente usadas, e ptp
não é uma delas.
Olhando para as funções nan*
, parece que a maioria das funções de base tem nan análogos. Eu concordo que ptp
não é a função mais comumente usada, mas eu a uso bastante em problemas de processamento de sinal e parece que está faltando devido às outras funções em nanfunctions.py
. Terminei de implementar uma versão básica, vou colocá-la na minha conta assim que for limpa e testada.
Se decidirmos seguir em frente com este, acabei de abrir um PR com a implementação da função e testes correspondentes.
Discutimos isso na reunião de triagem. A maioria das pessoas ficou feliz em rejeitar essa solicitação de recurso. @charris se absteve, @seberg disse que podia imaginar um namespace separado.
A principal razão aqui é que não queremos mais funções nan em geral; temos os mais comumente usados, e preencher o namespace principal com mais funções nan não se justifica (há um custo significativo para adicionar qualquer coisa ao namespace neste momento.
Em particular para este: ptp
já é um nicho e foi indiscutivelmente um erro adicionar a numpy. Portanto, nanptp
não é desejado.
Um pacote separado como bottleneck
faz mais sentido para mais funções nan.
Obrigado por sugerir @ Gabriel-p, e pelo PR @yliapis
Obrigado pela explicação detalhada @rgommers
Comentários muito úteis
Oi Gabriel, vou tentar no fim de semana