Привет, Габриэль, я займусь этим на выходных
Было бы неплохо, если бы это где-то было связано, но я думаю, что это слишком нише, чтобы включать в numpy. нам нужны только nan*
версии наиболее часто используемых функций, и ptp
не входит в их число.
Глядя на существующие функции nan*
, похоже, что у большинства базовых функций есть аналоги nan. Я согласен с тем, что ptp
не является наиболее часто используемой функцией, но я довольно часто использую ее для решения проблем с обработкой сигналов, и мне кажется, что она отсутствует, учитывая другие функции в nanfunctions.py
. Я завершил реализацию базовой версии, я зарегистрирую ее в своей учетной записи, как только она будет очищена и протестирована.
Если мы решим двигаться дальше с этим, я просто открыл PR с реализацией функции и соответствующими тестами.
Мы обсуждали это на совещании по сортировке. Большинство людей были счастливы отклонить этот запрос функции. @charris воздержался, @seberg сказал, что может представить себе отдельное пространство имен.
Основное объяснение здесь в том, что нам вообще не нужно больше нан-функций; у нас есть наиболее часто используемые, и заполнение основного пространства имен большим количеством нан-функций не оправдано (на данном этапе добавление чего-либо в пространство имен требует значительных затрат.
В частности, для этого: ptp
уже является нишевым, и, возможно, было ошибкой добавлять его в numpy. Итак, nanptp
нежелательно.
Отдельный пакет, например bottleneck
имеет больше смысла для большего количества нан-функций.
Спасибо за предложение @ Gabriel-p и за PR @yliapis
Спасибо за подробное объяснение @rgommers
Самый полезный комментарий
Привет, Габриэль, я займусь этим на выходных