Skip to content

Commit f413eac

Browse files
Merge pull request #184 from StingraySoftware/fix_empty_sequence_z9
Fix a bug coming up in Z with large n
2 parents be63b83 + 3ae5edd commit f413eac

1 file changed

Lines changed: 12 additions & 8 deletions

File tree

hendrics/efsearch.py

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1345,14 +1345,18 @@ def _analyze_qffa_results(input_ef_periodogram, fname=None):
13451345
)
13461346
max_stat = input_ef_periodogram.stat[f_idx, fdot_idx]
13471347
sig_e1_m, sig_e1 = power_confidence_limits(max_stat, c=0.68, n=input_ef_periodogram.N)
1348-
fmin, fmax, fdotmin, fdotmax = get_xy_boundaries_from_level(
1349-
input_ef_periodogram.freq,
1350-
input_ef_periodogram.fdots,
1351-
input_ef_periodogram.stat,
1352-
sig_e1_m,
1353-
f,
1354-
fdot,
1355-
)
1348+
if max_stat < sig_e1_m:
1349+
# Only add one candidate
1350+
fmin = fmax = fdotmin = fdotmax = np.nan
1351+
else:
1352+
fmin, fmax, fdotmin, fdotmax = get_xy_boundaries_from_level(
1353+
input_ef_periodogram.freq,
1354+
input_ef_periodogram.fdots,
1355+
input_ef_periodogram.stat,
1356+
sig_e1_m,
1357+
f,
1358+
fdot,
1359+
)
13561360
elif len(input_ef_periodogram.stat.shape) == 1:
13571361
f_idx = idx
13581362
allfreqs = input_ef_periodogram.freq

0 commit comments

Comments
 (0)