-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add Node of rou_fluo_from_hdf5 #575
base: develop-main
Are you sure you want to change the base?
Conversation
データの追加方法 Screen.Recording.2025-01-30.at.16.59.19.mov |
.filter((itemId) => { | ||
const item = selectVisualizeItemById(itemId)(state) | ||
return ( | ||
(isImageItem(item) && !item.isWorkflowDialog) || |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
細かい指摘ですが、この部分はrefactorできそうです。(!item.isWorkflowDialog の複数回判定が冗長)
|
||
# Enrich output with metadata and processed data | ||
output = { | ||
"iscell": IscellData(iscell_data, file_name="iscell") if iscell else None, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if iscell else None
の判定を複数回実施しているが、これは適切であるか?(冗長な処理、またはチェックすべき変数が適切ではないのでは?)
要件
ROI(領域)とFluorescence(蛍光データ)を同時に保持・確認できる仕組みを構築すること。
現状では、roi_from_hdf5 と fluo_from_hdf5 が別々に管理されており、同一データかどうかを確認することが困難な状況になっています。
対応内容
今後、HDF5内のROIとFluorescenceデータを連携して扱えるようにするため、これらを1つのノードで保持できる仕組みを作成しました。
このノードを利用することで、ROIとFluorescenceのデータを同時に管理・確認でき、データの連携が容易になります。
具体的な変更点:
ROIとFluorescenceの統合
連携対応
テストケース
https://docs.google.com/spreadsheets/d/1SXsn26sk67WvEVq4B3XGRTxK8mVkLPwpTNC9nz6bFnQ/edit?gid=1902535314#gid=1902535314