Ian Harry

Thanks @connor-mcisaac this is approved for merge, but it will need a rebase to pick up the fix for the failing test suite.

Hi Tom, thanks for doing this. I had a closer look at this, as I wouldn't expect this change to do anything (pypi will always take the larger release number...

The v16 release branch is now closed, so this should be closed as well ... We still don't have a minimum lalsuite requirement though, and might want one.

@SamuelH-97 For the docstrings I think you just need to add the "r" prefix: https://developer.lsst.io/python/numpydoc.html#docstrings-must-be-delimited-by-triple-double-quotes which should not mess up the latex. In this case I think it's okay as...

@santanu-tripathy This error (which we need to make more useful), normally indicates that you do not have necessary dependencies installed. The most likely culprit is that you don't have the...

@siangyuhuang Thanks for posting this issue here. Please do post any problems you might encounter here. We love seeing that folks are using PyCBC, and will do our best to...

Thanks @bhooshan-gadre ! I wanted to ask about how much duplication there is between the ambi chisq, and the existing bank chi-squared? If the additional functionality here is the ability...

On this point we did have some discussion with @idorrington92 when he was writing `pycbc_multi_inspiral`. At the moment `multi_inspiral` does support the single detector chi-squareds: https://github.com/gwastro/pycbc/blob/master/bin/pycbc_multi_inspiral#L390 but no form of...

No, it doesn't look like it.

Or at least half of it has. `pycbc_page_snglinfo` now does the right thing, but not `trigger_timeseries`