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

Add new sub-command for bagel-CLI derivatives: #345

Closed
11 of 13 tasks
Tracked by #342
surchs opened this issue Sep 24, 2024 · 1 comment · Fixed by #349
Closed
11 of 13 tasks
Tracked by #342

Add new sub-command for bagel-CLI derivatives: #345

surchs opened this issue Sep 24, 2024 · 1 comment · Fixed by #349
Assignees
Labels
released This issue/pull request has been released. type:feature Effort to deliver new features, feature changes & improvements

Comments

@surchs
Copy link
Contributor

surchs commented Sep 24, 2024

  • Inputs:
    • imaging bagel TSV from Nipoppy or TSV with same columns
      • add check that subject ID column has no missing values
    • JSONLD from bagel pheno command, OR pheno + bids command
      • Do we need a sanity check here that the file does not already contain derivative info?
    • output file path
    • override flag
  • Command should
    • Validate input columns from the bagel file against the Nipoppy bagel schema
    • Check that pipeline names and versions are part of allowed terms
    • Check that all subject IDs from the derivatives TSV are represented in the existing JSONLD, otherwise error
    • If there are imaging sessions in the JSONLD, try to match them to sessions from the derivatives TSV
    • If there are new sessions in the derivatives TSV, create new imaging sessions
    • If a session ID is missing, create a new imaging session with a custom session label
    • Extract pipeline names for a given subject-session entry in the bagel TSV, and also the completion status
@surchs surchs added feat:add type:feature Effort to deliver new features, feature changes & improvements labels Sep 24, 2024
@surchs surchs moved this to Specify - Done in Neurobagel Sep 24, 2024
@alyssadai alyssadai self-assigned this Sep 26, 2024
@github-project-automation github-project-automation bot moved this from Review - Active to Review - Done in Neurobagel Oct 8, 2024
Copy link
Contributor

🚀 Issue was released in v0.3.0 🚀

@neurobagel-bot neurobagel-bot bot added the released This issue/pull request has been released. label Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released This issue/pull request has been released. type:feature Effort to deliver new features, feature changes & improvements
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants