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

sanity checks for TMA remapping #16

Closed
ngreenwald opened this issue Jan 15, 2022 · 1 comment · Fixed by #30
Closed

sanity checks for TMA remapping #16

ngreenwald opened this issue Jan 15, 2022 · 1 comment · Fixed by #30
Labels
enhancement New feature or request

Comments

@ngreenwald
Copy link
Member

ngreenwald commented Jan 15, 2022

In order to help prioritize user curation, we should create a couple distinct helper functions which check for likely errors in mapping.

  1. Distance between mapped centroids: This function will compute the distance between the centroids of each manual/automatic pair. It will then return the names of the 'x' FOVs with the greatest centroid distance, with the threshold set by the user.
  2. Duplicate mapping: This function will return the names of any manual FOVs which map to the same automatic FOVs
  3. Exact matching: This function will return the names of any manual FOVs whose name doesn't exactly match the name of the automatic FOV they are mapped to
@ngreenwald
Copy link
Member Author

@alex-l-kong I updated the description for what to do for the mapping validation function

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant