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

Merge dev branch to main #96

Merged
merged 4 commits into from
Aug 30, 2021
Merged

Merge dev branch to main #96

merged 4 commits into from
Aug 30, 2021

Conversation

sidward
Copy link
Collaborator

@sidward sidward commented Aug 30, 2021

Summary of main changes:

  • Fix coil looping bug in SENSE linop.
  • Add upgrade option to pip install to let Travis install numpy.

sidward and others added 4 commits August 18, 2021 18:22
Fix: When using coil_batch_size, SENSE linop stacked coil sensitivity maps interleavedly, which does not work properly for coil_batch_size > 1. Now, SENSE linop stacks them successively.
Test: Modified function in test_app (test_shepp_logan_SenseRecon) and modified two in test_linop (test_sense_model_batch and test_noncart_sense_model_batch) for coil_batch_size >= 1.
Fix SENSE linop to work with coil_batch_size > 1 + Minor change to pip install for travis
@sidward sidward merged commit dde242d into master Aug 30, 2021
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.

1 participant