Skip to content

Conversation

mattcieslak
Copy link
Collaborator

@mattcieslak mattcieslak commented Apr 1, 2025

Closes #6

Adds simbids-raw-mri CLI for making simulated BIDS mri data

@mattcieslak mattcieslak requested a review from tsalo April 1, 2025 13:34
pyproject.toml Outdated
Comment on lines 87 to 88
[tool.hatch.build.targets.wheel.include]
"simbids/data/**/*.yaml" = "simbids/data"
Copy link
Member

Choose a reason for hiding this comment

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

Might be good to make an alpha release first in case this breaks the deployment like the similar field did in BABS.

Copy link
Member

@tsalo tsalo left a comment

Choose a reason for hiding this comment

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

This looks good to me. From a design perspective, one thing that jumps out to me is that we should probably merge parser.py and run.py as something like derivative_mri.py since there will be two equally important CLIs. That can wait until a later PR though.

@mattcieslak mattcieslak merged commit 8eeaea6 into main Apr 1, 2025
2 of 9 checks passed
@mattcieslak mattcieslak deleted the create-raw-cli branch April 1, 2025 15:04
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.

Create CLI entrypoint for creating raw bids data
2 participants