search

openEXO / cloud-kepler

Cloud Kepler is a cloud enabled Kepler Planet searching pipeline
8 stars 3 forks source link

Determine why bls_pulse is missing a few transits in Unit Test #18

Closed scfleming closed 10 years ago

scfleming commented 10 years ago

bls_pulse is currently misisng a few transits, generally from the shorter-period test cases. I suspect this is due to poor choice of segment size and/or number of bins per segment. Need to investigate this to see if I can get something that will work for all transits in all 10 test cases in the Unit Test.

scfleming commented 10 years ago

This was primarily due to very poor choices of input parameter, but I also found some bugs. Now I believe all transits are, in fact, being found, although I need to improve the pass/fail criteria a bit. Closing this for now and making that a separate Issue.