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

what should be the reference NFFT backend #19

Closed
paquiteau opened this issue Jul 26, 2023 · 1 comment
Closed

what should be the reference NFFT backend #19

paquiteau opened this issue Jul 26, 2023 · 1 comment
Labels
nufft-backend Issues regarding NUFFT backend support

Comments

@paquiteau
Copy link
Member

Currently we are using pyNFFT as the reference backend to check the fidely of the other supported backend.
This result from a speed/precision compromise, because comparing all backends to the NDFT would be expensive

However this causes some troubles:

An alternative would be to use finufft as the reference implementation, as they have prime support for python bindings.

@ghisvail
Copy link

ghisvail commented Jul 31, 2023

👋 PyNFFT author here

Indeed, I wrote the package a while back but the engagement around it has faded.

Feel free to submit a patch for Cython 3.0 if you succeed in making it work with it.

It's okay if you find an alternative implementation which performs better and is actively maintained.

Best regards from a fellow Inria engineer.

@paquiteau paquiteau added the nufft-backend Issues regarding NUFFT backend support label Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
nufft-backend Issues regarding NUFFT backend support
Projects
None yet
Development

No branches or pull requests

2 participants