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

Site-specific adjustments for 3D-EPI SE/STE B1 mapping #16

Open
antoinelutti opened this issue Sep 24, 2020 · 2 comments
Open

Site-specific adjustments for 3D-EPI SE/STE B1 mapping #16

antoinelutti opened this issue Sep 24, 2020 · 2 comments
Assignees
Labels
bug Something isn't working enhancement New feature or request futureplans

Comments

@antoinelutti
Copy link

Retrieved acquisition settings are dependent on potentially non-unique sequence labels
This needs to be resolved before pull request #13 can be resolved.

@antoinelutti antoinelutti added bug Something isn't working enhancement New feature or request labels Sep 24, 2020
@lukeje lukeje assigned kerrinpine and unassigned lukeje Oct 23, 2020
@lukeje lukeje changed the title Site-specific adjustments Site-specific adjustments for 3D-EPI B1 mapping Jul 19, 2022
@lukeje
Copy link
Member

lukeje commented Jul 19, 2022

I think the BIDS fields for 3D-EPI SE/STE B1 mapping are relevant to this issue, as they specify the fields that we should try and populate: https://bids-specification.readthedocs.io/en/stable/99-appendices/11-qmri.html#tb1epi-specific-notes

@lukeje lukeje changed the title Site-specific adjustments for 3D-EPI B1 mapping Site-specific adjustments for 3D-EPI SE/STE B1 mapping Jul 19, 2022
@lukeje
Copy link
Member

lukeje commented Jan 30, 2023

Based on a suggestion from @ChristophePhillips: we could add an additional step between dicom conversion and map creation which replaces empty fields in a BIDS-style sidecar file needed for calculation with "default" values specified in a configuration file. This would allow us to remove a lot of complicated code from the map creation module as this will promise that fields will exist.

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

No branches or pull requests

5 participants