Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Edit ROI with Caiman and LCCD #453

Merged
merged 25 commits into from
May 6, 2023
Merged

Edit ROI with Caiman and LCCD #453

merged 25 commits into from
May 6, 2023

Conversation

quanpython
Copy link
Collaborator

No description provided.

@quanpython quanpython marked this pull request as ready for review April 28, 2023 08:50
@emuemuJP emuemuJP requested a review from ReiHashimoto April 28, 2023 09:34
@ReiHashimoto
Copy link
Collaborator

  • ROIのMerge, Deleteの際にもdisableになっているアクションのリンクのopacityを変更するstyleを追加しました

@ReiHashimoto
Copy link
Collaborator

確認事項概要

  • CaImAn, LCCDそれぞれについてROIの編集(add, merge, delete)に対応している
  • add、mergeされたROIはindexが末尾に追加される

検証

実行環境

  • MacOS 13.3.1, Apple M2
  • バックエンドはpython main.py、フロントエンドはyarn startで起動し、3000番ポートでブラウザから利用

作業内容

  • 以下のワークフローでcaiman_cnmfとlccdを実行
    workflow

  • caiman_cnmf、lccdのcell_roisそれぞれに対して、add, merge, delete処理を実行

  • RECORDタブを開きRELOADボタン押下後にcaiman_cnmf, lccdのnwbファイルを取得

結果

以下の通り、概要の要件が満たされていることを確認

図は左からcaiman_cnmf, lccd

@emuemuJP
Copy link
Collaborator

emuemuJP commented May 2, 2023

@ReiHashimoto HDFView: https://www.hdfgroup.org/downloads/hdfview/?1682476202
を使ってNWBファイルの中にadd roi, delete roi, merge roiしたIDが保存されているか、確認できますか?

@emuemuJP
Copy link
Collaborator

emuemuJP commented May 2, 2023

こちらにOptinistにより作成されるNWBの構造が書かれています。まだ書きかけですが。
https://github.com/oist/optinist/blob/develop-main/docs/utils/nwb_file.md

@ReiHashimoto
Copy link
Collaborator

@emuemuJP

@ReiHashimoto HDFView: https://www.hdfgroup.org/downloads/hdfview/?1682476202 を使ってNWBファイルの中にadd roi, delete roi, merge roiしたIDが保存されているか、確認できますか?

Screenshot 2023-05-02 at 16 05 01

図のようにデータが格納されていることを確認しました。
add, deleteは操作対象のidと一致しており、
mergeはmerge後のid, mergeされたid, ... , -1.0の形式です。
mergeの-1.0は期待の挙動でしょうか?

@emuemuJP
Copy link
Collaborator

emuemuJP commented May 2, 2023

@ReiHashimoto はい、ちょっと仕様に書けてないですが、
merge後ID, merge前ID 1, ..., merge前ID N, -1.0 , merge後ID, merge前ID 1, ..., merge前ID N, -1.0 ,
の様な形でセパレータとして-1.0を入れています

Copy link
Collaborator

@ReiHashimoto ReiHashimoto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@emuemuJP emuemuJP merged commit cdcdf57 into develop-main May 6, 2023
quanpython pushed a commit that referenced this pull request Jun 28, 2023
Edit ROI with Caiman and LCCD
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants