We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
mri-nufft/src/mrinufft/operators/interfaces/tfnufft.py
Lines 48 to 51 in 8e15f05
Here, estimate_density just estimates the density, not the density_compensators. What we need is a reciprocal of this values.
estimate_density
density_compensators
All we need to do is to change this line to: tf.math.reciprocal_no_nan(tfmri.estimate_density( samples, shape, method="pipe", max_iter=15 ))
Also, some tests on this would be helpful.
The text was updated successfully, but these errors were encountered:
Lenoush
Successfully merging a pull request may close this issue.
mri-nufft/src/mrinufft/operators/interfaces/tfnufft.py
Lines 48 to 51 in 8e15f05
Here,
estimate_density
just estimates the density, not thedensity_compensators
. What we need is a reciprocal of this values.All we need to do is to change this line to:
tf.math.reciprocal_no_nan(tfmri.estimate_density(
samples, shape, method="pipe", max_iter=15
))
Also, some tests on this would be helpful.
The text was updated successfully, but these errors were encountered: