TeamXenonDuke / xenon-gas-exchange-consortium

0 stars 0 forks source link

Implement automatic complex conjugate for GE platform, and proper orientation for all 3 platforms #59

Closed davidmummy closed 2 months ago

davidmummy commented 3 months ago

When reading in the MRD, please check the scanner type. If it is GE, apply a complex conjugate to the data. Otherwise, do nothing.

This mirrors the functionality currently in our spectroscopy pipeline as well.

sl609 commented 3 months ago

Add Branch to fix the issue

davidmummy commented 3 months ago

Update -- we will also implement the vendor-specific image orientation (#57 ) as part of this issue, since the complex conjugate is tied up in the orientation for GE scanners. Here is the spec for that one:

Different vendors write their data differently to the MRD file, resulting in the orientation of images having different coordinate systems after processing.

Desired functionality:

This is ready to work at your convenience.

davidmummy commented 2 months ago

Assigning to myself to review.

davidmummy commented 2 months ago

Tested on MRDs from Duke subject 009-024A and Cincy ILD-HC-068_V2. Everything looks good and properly oriented.

Have requested updated MRD from Iowa to complete the collection. Will update as soon as I get it and then I think this will be ready to close.