Numpy: [Запрос функции] Добавить функцию nanptp

Созданный на 27 мар. 2019  ·  6Комментарии  ·  Источник: numpy/numpy

Есть функция ptp, но она не сработает, если в массиве есть значения nan .

Самый полезный комментарий

Привет, Габриэль, я займусь этим на выходных

Все 6 Комментарий

Привет, Габриэль, я займусь этим на выходных

Было бы неплохо, если бы это где-то было связано, но я думаю, что это слишком нише, чтобы включать в numpy. нам нужны только nan* версии наиболее часто используемых функций, и ptp не входит в их число.

Глядя на существующие функции nan* , похоже, что у большинства базовых функций есть аналоги nan. Я согласен с тем, что ptp не является наиболее часто используемой функцией, но я довольно часто использую ее для решения проблем с обработкой сигналов, и мне кажется, что она отсутствует, учитывая другие функции в nanfunctions.py . Я завершил реализацию базовой версии, я зарегистрирую ее в своей учетной записи, как только она будет очищена и протестирована.

Если мы решим двигаться дальше с этим, я просто открыл PR с реализацией функции и соответствующими тестами.

13220

Мы обсуждали это на совещании по сортировке. Большинство людей были счастливы отклонить этот запрос функции. @charris воздержался, @seberg сказал, что может представить себе отдельное пространство имен.

Основное объяснение здесь в том, что нам вообще не нужно больше нан-функций; у нас есть наиболее часто используемые, и заполнение основного пространства имен большим количеством нан-функций не оправдано (на данном этапе добавление чего-либо в пространство имен требует значительных затрат.

В частности, для этого: ptp уже является нишевым, и, возможно, было ошибкой добавлять его в numpy. Итак, nanptp нежелательно.

Отдельный пакет, например bottleneck имеет больше смысла для большего количества нан-функций.

Спасибо за предложение @ Gabriel-p и за PR @yliapis

Спасибо за подробное объяснение @rgommers

Была ли эта страница полезной?
0 / 5 - 0 рейтинги