GitLab

Renames sftfilepath -> sftfilepattern

See merge request !6

This renames the input sftfilepath to sftfilepattern and adds
documentation on how that should be used, i.e. a colon separated list of
wildstring or exact matches. In globbing for all matches, the colon
split is added in. sftfilepath is still used by `Writer` since
an exact path is known.

Gregory Ashton authored Jul 17, 2017 and Gregory Ashton committed Jul 17, 2017

Previously, each search class named its own output file. This provides a
function `set_out_file` in the top-level class which sets the `out_file`
attribute to be used.
- Adds class name to the out_file for clarity
- Add detectors used

Note - conflict resolved whilst cherry-picking

Fix sftfilenames and sftfilepath for multiple detectors

See merge request !5

Gregory Ashton authored Jul 17, 2017 and Gregory Ashton committed Jul 17, 2017

If there are multiple detectors, then multiple .sft files are generated.
This updates the attribute 'sftfilename' -> 'sftfilenames', a list of
those files while also updating 'sftpilpath' to a colon separated list
of the relevant sft files.

Previously, each search class named its own output file. This provides a
function `set_out_file` in the top-level class which sets the `out_file`
attribute to be used.
- Adds class name to the out_file for clarity
- Add detectors used

If there are multiple detectors, then multiple .sft files are generated.
This updates the attribute 'sftfilename' -> 'sftfilenames', a list of
those files while also updating 'sftpilpath' to a colon separated list
of the relevant sft files.

- In a prior commit, the number of sidebands checked for the normal search
(i.e. SSBprec=2) was erroneously set to M0, rather than 9. This fixes
that.
- Also improves logging output
- Splits of the UniformPriorToGrid run function to produce the data and
  plot separately

The command line argument is matched to formatting used by MFD to
reconstruct the argument (using .16g) formatting. Ideally, all REAL8
should use this formatting.

Removes the forced `int` for `fmin` in MFD. As noted by David, this
isn't required.

add SFT-vs-output timestamp check to gridsearch, and fix bug in _get_list_of_matching_sfts

See merge request !4

new helper function run_commandline()

See merge request !3

 -ported check from MCMC search
 -bug: for single-str SFT path, glob list comprehension
  was trying every single character

 -try-except wrapper for subprocess.check_output()
 -use this for MFDv5 call (properly terminate at error!)
 -and for other small utility calls
 -use unique variable names for cl strings too

workaround for matplotlib on X-less remote logins

See merge request !2

 -first check $DISPLAY environment variable
 -then use non-interactive "AGG" backend if necessary

peakutils missing in requirements

See merge request !1

- Adds ability to use either a dict or a par file
- Fixes bug in which if tstart and tend aren't specificed the data
  saving mechanism breaks

- Add peak detection utilities
- Move generate_loudest
- Add SSBprec pass through to MCMC searches
- Fix convergence stat (missing sqrt)
- Minor fixes to plot_walkers (put detection statistic behind walkers)

- Adds injectSources to a variety of other methods
- Adds conversion of injectSource to pfs_input

- Uses the actual prior and not the log-prior as previous
- Adds option to plot the full extent (currently only for uniform
  priors)
- ensures the xlims and ylims are reset correctly

If the given tstart is before the minStartTime this will error - so
instead, select the minStartTime