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.
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
Komentar yang paling membantu
Hai Gabriel, saya akan mencoba ini selama akhir pekan