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

prepfold "amount complete" above 100% #99

Open
wcfiore opened this issue May 24, 2019 · 10 comments
Open

prepfold "amount complete" above 100% #99

wcfiore opened this issue May 24, 2019 · 10 comments

Comments

@wcfiore
Copy link

wcfiore commented May 24, 2019

I'm performing a fairly standard prepfold command-
prepfold -nsub 128 -mask guppi_57955_P86Y1272_0003_rfifind.mask -p 0.00246068325 -dm 33.08 -searchpdd -o joecandsearchpdd *.fits
After it's done folding it searches over DMs, periods, pdots, and pdotdots as normal, but when "amount complete" reached 100% it didn't stop. It's currently at 145% and climbing...

I'll note I performed this exact same prepfold command sans the -searchpdd flag and it worked just fine.

@scottransom
Copy link
Owner

scottransom commented May 24, 2019 via email

@wcfiore
Copy link
Author

wcfiore commented May 24, 2019

Alright, thanks! Agreed, it's quite strange but it does make sense that searching over too many parameters would cause issues.

@scottransom
Copy link
Owner

I'll still check into it. Thanks for making an issue!

@parulj3795
Copy link

Hi, I ran the following command after finding the DM and period from the cands.txt file -
prepfold -n 512 -p 0.005757491 -dm 3.0 J4037.fil
The amount complete went up to 890% and didn't stop.

@scottransom
Copy link
Owner

Hmmm. This is really bizarre. I can't reproduce this with data that I have. I'm trying it with the same exact types and numbers of arguments as well as with an input .fil file. It is really bizarre to me.

@parulj3795
Copy link

J4037.fil is a typo, it is J0437.fil. The similar command works for other data set (for different pulsar). But even in this case, I am able to get a profile if I use -nodmsearch.

prepfold -n 512 -p 0.005757491 -nodmsearch -dm 3.0 J4037.fil

@keegansmith21
Copy link

Hi, I'm trying to execute the command:

prepfold -psr J0255-5304 -o 1224859816_1024_bins -n 1024 -dmstep 1 -npart 120 -npfact 1 -ndmfact 1 -ncpus 8 -dm 17.961 -p 447.723924479956 *.fits -pstep 1 -pdstep 2 -noxwin -runavg -noclip -nsub 256

And i'm having the same issue. Prepfold is trying to search over 2049 dm, p, pd parameters but the search % keeps rising. Strangely, this works fine when i fold over 100 bins instead.

@scottransom
Copy link
Owner

@keegansmith21 I looking into this right now again and am worried that I won't be able to trigger the issue. But we'll see.

I do have a couple of comments about the command line, though:

  1. If you use the command line option "-fine", it sets -ndmfact 1 -dmstep 1 -npfact 1 -pstep 1 -pdstep 2 all at once.
  2. -noclip is rarely what you want for radio data unless you are looking at a pulsar with very small amounts of DM sweep across the band. It is quite good at getting rid of impulsive RFI and not messing with your pulsar signal
  3. -ncpus greater than 1 doesn't really do anything very useful at present. Working on the OpenMP capabilities of PRESTO is definitely on my ToDo list
  4. If you use -psr, you will get the ATNF values for folding a pulsar, but you are also using -p and specifcying it at 447 seconds! If you wanted to specify the pulsar period, don't give -psr and a name, just use -p in seconds (you should be using 0.4477...)
  5. -runavg is kind of dangerous for slow pulsars as you can end up removing some of the signal of your pulsar (since it is a form of high-pass filter). However, sometimes it is useful if the power levels in an observation change a lot

Hope these tips help. And I'll let you know if I can figure out what is causing the search to go above 100%

@scottransom
Copy link
Owner

@keegansmith21 and @parulj3795 I think I just fixed this issue with the latest commit (d365d2d). Basically, if you are searching over too many trails in DM, P, Pdot, and/or Pdotdot space, the integer holding the number of trials overflowed! And when that got turned into a float, there also wasn't enough precision! The new fixes actually tell you how many trials you will really search and will give you a Warning (or a really stern warning) if your number of trials is >1e8 (>1e9).

Please test this and let me know if it helps! Also, using lots and lots of bins in the profiles makes the number of trials a lot bigger. So if you do that, definitely try to use at least one of -nosearch, -nopsearch, -nopdsearch, or -nodmsearch.

@keegansmith21
Copy link

keegansmith21 commented Feb 21, 2020

Thanks Scott, I'll try out the fix and let you know if it's still an issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants