Numpy: [Permintaan fitur] Tambahkan fungsi nanptp

Dibuat pada 27 Mar 2019  ·  6Komentar  ·  Sumber: numpy/numpy

Ada fungsi ptp tetapi akan gagal ketika ada nilai nan dalam array.

Triaged

Komentar yang paling membantu

Hai Gabriel, saya akan mencoba ini selama akhir pekan

Semua 6 komentar

Hai Gabriel, saya akan mencoba ini selama akhir pekan

Memiliki tautan ini di suatu tempat akan menyenangkan, tetapi saya pikir itu terlalu khusus untuk dimasukkan dalam numpy. kami hanya menginginkan versi nan* dari fungsi yang paling umum digunakan, dan ptp bukan salah satunya.

Dari melihat fungsi nan* , sepertinya sebagian besar fungsi dasar memiliki analog nan. Saya setuju bahwa ptp bukan fungsi yang paling umum digunakan, tetapi saya menggunakannya sedikit pada masalah pemrosesan sinyal, dan rasanya seperti hilang mengingat fungsi lain di nanfunctions.py . Saya telah selesai mengimplementasikan versi dasar, saya akan memasangnya di akun saya setelah dibersihkan dan diuji.

Jika kami memutuskan untuk melanjutkan dengan yang ini, saya baru saja membuka PR dengan implementasi fungsi dan tes yang sesuai.

13220

Kami membahas ini dalam pertemuan triase. Kebanyakan orang senang dengan menolak permintaan fitur ini. @charris abstain, @seberg mengatakan dia bisa membayangkan namespace yang terpisah.

Alasan utama di sini adalah bahwa kita tidak menginginkan lebih banyak fungsi nan secara umum; kami memiliki yang paling umum digunakan, dan mengisi namespace utama dengan lebih banyak fungsi nan tidak dibenarkan (ada biaya yang signifikan untuk menambahkan apa pun ke namespace pada saat ini.

Khususnya untuk yang ini: ptp sudah menjadi niche, dan bisa dibilang kesalahan untuk ditambahkan ke numpy. Jadi nanptp tidak diinginkan.

Paket terpisah seperti bottleneck lebih masuk akal untuk lebih banyak fungsi nan.

Terima kasih telah menyarankan ini @Gabriel-p, dan untuk PR @yliapis

Terima kasih atas penjelasan detailnya @rgommers

Apakah halaman ini membantu?
0 / 5 - 0 peringkat

Masalah terkait

keithbriggs picture keithbriggs  ·  3Komentar

kevinzhai80 picture kevinzhai80  ·  4Komentar

dmvianna picture dmvianna  ·  4Komentar

ghost picture ghost  ·  4Komentar

manuels picture manuels  ·  3Komentar