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

Overhaul of the Reconstruction Pipeline to use a scanobj #164

Merged
merged 9 commits into from
Apr 24, 2024

Conversation

timwahoo
Copy link
Contributor

Fixes #160

Changes proposed in this pull request:

  • The entire recon.py file and recoFunction.py file has been rewritten to minimize passing large objects around
  • recoSigpy.py file has been removed to not require SigPy as a brkraw requirement. The motivation for this move is to establish brkraw as an INTERMEDIATE for image reconstruction methods in MIRT, SigPy, and BART. A wrapper function and tutorial will added later for SigPy and BART.
  • recon.py now contains a single class that takes a scanObj, certain acqp, meth, and reco variables necessary for FID sorting and simple image reconstruction. Users will have to option to either take the root mean square (rms) of the complex image.
  • The Reconstruction object can return 3 numpy objects (readout sorted FID, processed kspace, and image reconstruction using a simple Fourier transformation. Compressed Sensing will return a zerofilled sorted kspace matrix
  • Please look into recon_api_test.py for an example on how to create a nifti file from the reconobj.

@BrkRaw/Bruker

@dvm-shlee dvm-shlee merged commit 61559de into BrkRaw:main Apr 24, 2024
26 checks passed
@timwahoo timwahoo deleted the recon_api branch April 24, 2024 16:02
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.

Simplifying and Reducing brkrecon submodule
2 participants