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

Change 'AccelerationFactor' to 'ParallelAcquisitionTechnique' #7

Closed
AdebayoBraimah opened this issue Dec 6, 2019 · 3 comments
Closed
Assignees
Labels
documentation Improvements or additions to documentation enhancement New feature or request high effort requires significant effort high impact high impact or significant improvement

Comments

@AdebayoBraimah
Copy link
Owner

Change AccelerationFactor to ParallelAcquisitionTechnique in the JSON side car. This element should report a string that yields the name of parallel acquisition type (e.g. GRAPPA or SENSE).

The current (float) value that is reported in the AccelerationFactor field should be renamed to ParallelReductionFactorInPlane.

see reference(s) for details:
https://bids-specification.readthedocs.io/en/latest/04-modality-specific-files/01-magnetic-resonance-imaging-data.html
https://github.com/bids-standard/bids-specification/blob/master/src/04-modality-specific-files/01-magnetic-resonance-imaging-data.md

@AdebayoBraimah
Copy link
Owner Author

N.B. SENSE is used primarily for Philips - and is currently found by a regex search of the filename and/or certain parts of the PAR header file. Currently unsure if this parameter is report in Philips DICOM header.

@AdebayoBraimah AdebayoBraimah self-assigned this Dec 6, 2019
@AdebayoBraimah AdebayoBraimah added documentation Improvements or additions to documentation enhancement New feature or request labels Dec 6, 2019
@AdebayoBraimah
Copy link
Owner Author

AdebayoBraimah commented Dec 6, 2019

Should also add feature that allows for one to specify scanner type - as GRAPPA is the general term.
Vendors have specific names for each of their own parallel acquisition/reconstruction techniques.

see reference for further details:
http://mriquestions.com/senseasset.html

@AdebayoBraimah AdebayoBraimah added high effort requires significant effort high impact high impact or significant improvement labels Dec 6, 2019
@AdebayoBraimah
Copy link
Owner Author

Added the functionality, but it is not accessible. The corresponding fields in the JSON sidecar have been added/updated. These changes will be reflected in the next major release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request high effort requires significant effort high impact high impact or significant improvement
Projects
None yet
Development

No branches or pull requests

1 participant