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

adding spot detection process name #97

Merged
merged 4 commits into from
Nov 25, 2024
Merged

Conversation

camilolaiton
Copy link
Contributor

Hello, @saskiad

I am going to generate metadata for some of the HCR pipeline steps and I need to include spot detection process name. There are many names for this process, another one would be Image blob detection (which is more computer vision related, Image spot detection is mostly in microscopy).

Let me know if you prefer something else.

@camilolaiton camilolaiton requested review from dyf and saskiad October 31, 2024 20:15
@@ -15,6 +15,8 @@ Image background subtraction
Image cell classification
Image cell quantification
Image cell segmentation
Image cross-round alignment
Image cross-session alignment
Copy link
Collaborator

Choose a reason for hiding this comment

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

what's the difference between cross-round and cross-session? My guess is that cross-round is on the HCR side and cross-session is on the ophys session. But do they really need distinct names - it's the same concept?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@saskiad I think you're right. Just discuss this with the team since I am not very familiar with ophys, the only difference is that cross-round is for 3D images of the same tissue for round to round registration and cross-session is for 2D image alignment. Would Image cross-image alignment be a better option?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @saskiad, please let me know what you think. Thanks!

@saskiad saskiad merged commit c75debc into main Nov 25, 2024
4 checks passed
@saskiad saskiad deleted the feat-spot-detection-process branch November 25, 2024 19:32
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.

2 participants