-
Notifications
You must be signed in to change notification settings - Fork 177
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
rfifind
and OpenMP
#175
Comments
Hey Wael, You are completely correct (unfortunately). I added the command line options and included the appropriate headers for OpenMP many years ago in anticipation of a big parallel push. But my initial experiments with speed-ups were terrible. I think it is because I need to restructure how the code collects and merges the "birdies" after running Unfortunately I haven't had the time to prioritize this work, though. So if you or a student wanted to tackle it, I think it would make a really nice computational project. A related addition I've been thinking about is adding generalized spectral kurtosis estimates... Let me know if you have any thoughts as I'd love to move forward on this. Scott |
Hi Scott, Thanks for the response! I guess it will be great to have In the meantime,
Is there any command line argument I can add that removes some of the processing to make it run faster? I am searching for FRBs, so probably won't need any FFT-based masking. -- Wael |
What I usually do in situations like this is only worry about the bad channels and forget about the time-variable stuff. To do that, I would use just a small chunk of the data (maybe 5-10%?) and run The ignorechan stuff is mentioned in the tutorial and also here: |
Just for reference, I should have put a link in for the FAQ entry about the lack of multi-CPU speed-up, in case others come here looking for answers: |
Hi @scottransom
It seems to me like
rfifind
does not utilize any parallelism. I tried the command line with and without-ncpus 6
and it didn't make a difference in runtime. I still getUsing 6 threads with OpenMP
when I pass-ncpus 6
.In the source code, I see this:
https://github.com/scottransom/presto/blob/master/src/rfifind.c#L121
But I don't see openMP calls or
#pragma
directives anywhere else.Am I missing something?
Thanks!
The text was updated successfully, but these errors were encountered: