Numpy: [Solicitação de recurso] Adicionar função nanptp

Criado em 27 mar. 2019  ·  6Comentários  ·  Fonte: numpy/numpy

Existe uma função ptp , mas ela falhará quando houver nan valores na matriz.

Triaged

Comentários muito úteis

Oi Gabriel, vou tentar no fim de semana

Todos 6 comentários

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.

13220

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

Esta página foi útil?
0 / 5 - 0 avaliações

Questões relacionadas

amuresan picture amuresan  ·  4Comentários

kevinzhai80 picture kevinzhai80  ·  4Comentários

keithbriggs picture keithbriggs  ·  3Comentários

astrofrog picture astrofrog  ·  4Comentários

Levstyle picture Levstyle  ·  3Comentários