-
Notifications
You must be signed in to change notification settings - Fork 9
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
Update the trajectory and DCp when self.samples is updated #132
Comments
Please add on if theres anything I missed. Generally I would also prefer sensitivity maps to change. But thats a different ballgame as the sensitivity maps need to anyway change with k-space data also. |
This was referenced Jun 21, 2024
All of this is handeled now! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This issue will track a bunch of features needed for learning k-space sampling trajectories.
For now, what we need is:
self.samples
property setter to have update the corresponding plans to update the trajectory in respectiveraw_op
for each backend.finufft
andcufinufft
easily as there is a set_pts function. @alineyyy will handle a PR for it. @alineyyy for other backends, please add an error for now. This can be done by having error raised in the main setter in the base class.gpuNUFFT
, we need some update ongpuNUFFT
side. @chaithyagr will handle this. Add support for updating points with gpuNUFFT #133The text was updated successfully, but these errors were encountered: