pocketfft icon indicating copy to clipboard operation
pocketfft copied to clipboard

Is there a pthreads pool available that works on Windows?

Open carlkl opened this issue 4 years ago • 8 comments

see https://github.com/mreineck/pocketfft/issues/1#issuecomment-977757283 and https://github.com/mreineck/pocketfft/issues/1#issuecomment-977839096 Related msys2 scipy issue: https://github.com/msys2/MINGW-packages/issues/10164

Is pocketfft with thread support functional on windows? And if not, what has to be done to enable this functionality?

carlkl avatar Nov 24 '21 12:11 carlkl

I have not heard of problems with pocketfft on Windows so far, and my own CI tests for the successor package (https://github.com/litebird/ducc) seem to work fine with multiple threads on Windows. However those tests are using MSVC and do not require MinGW.

If someone could help me set up these tests for MinGW using Github Actions, I'm happy to do further analysis!

mreineck avatar Nov 24 '21 12:11 mreineck

@mreineck, this is a nice reference project for our question. If will check, if this works with msys2 g++ as well.

carlkl avatar Nov 24 '21 12:11 carlkl

Very good! The multi-threading part has been moved out of the FFT code in this project, but it is essentially hte same as in pocketfft (just with some more functionality put on top). I'm sure that if we can get one to work, we'll manage with the other as well.

mreineck avatar Nov 24 '21 13:11 mreineck

@mreineck, are there some simple test cases available as well?

carlkl avatar Nov 24 '21 15:11 carlkl

ups, found it in the repo...

carlkl avatar Nov 24 '21 15:11 carlkl

ducc0 binary wheels (Python-3.9 64bit) are available here for some time: https://my.hidrive.com/share/7gun0qvp2v

carlkl avatar Nov 25 '21 10:11 carlkl

It might help if you could grab some stack traces of where it hangs. However, If the bug really is in the compiler and/or system libraries then I'm not sure what we could do.

peterbell10 avatar Nov 25 '21 16:11 peterbell10

see https://github.com/msys2/MINGW-packages/issues/10164#issuecomment-979823333

carlkl avatar Nov 26 '21 09:11 carlkl