search

KamitaniLab / bdpy

Python package for brain decoding analysis (BrainDecoderToolbox2 data format, machine learning analysis, functional MRI)
MIT License
33 stars 22 forks source link

select_topのnan対策 #46

Open micchu opened 1 year ago

micchu commented 1 year ago

https://github.com/KamitaniLab/bdpy/blob/f2db58a36bf1544cae41a8bbfa0e9a3ec81d9b68/bdpy/preproc/select_top.py#L41

valueにnanが入っているとargsortで後ろに回されたたnanがtopに来るという微妙な挙動になるので，修正しておいていいですか？ この行の直前で，value内のnanを-np.infで置き換えるつもりです．

ganow commented 1 year ago

（このissue対応済みに見えるのでcloseしてよいですかと聞こうと思いましたが） select_top(data, value) の期待する挙動として、そもそも value に nan が来ることを想定していない気がするので、 nan が入っていた場合 warning を表示するなり exception で落とすなりするほうがよいのではないかと思いました。