Skip to content

add script for creating folder with difficult cases #132

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

naga-karthik
Copy link
Collaborator

This PR adds a script to create a list of difficult cases. This list of subjects could be used for benchmarking our segmentation models.

Resolves #131

@naga-karthik naga-karthik marked this pull request as ready for review January 27, 2025 20:13
Copy link
Member

@jcohenadad jcohenadad left a comment

Choose a reason for hiding this comment

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

i would change the name of the function-- it is misleading-- eg: create_folder_of_difficult_subjects.py

Copy link
Member

@jcohenadad jcohenadad left a comment

Choose a reason for hiding this comment

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

i don't think this script will be very useful in the long term-- the idea is:

  • for each project (in this case, contrast-agnostic), identify the subjects based on:
    • BIDS dataset root and subj (eg: data/xxx/sub-xxx)
    • concatenate those into a YAML that goes as input into sct_run_batch
    • in the future we can run sct_run_batch -include-list
  • What if subjects span across multiple BIDS datasets?

@naga-karthik naga-karthik added the wontfix This will not be worked on label Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

create list of problematic subjects used for qualitative testing
2 participants