search

mikekatz04 / BBHx

GNU General Public License v3.0
20 stars 16 forks source link

Mode order in documentation plots #15

Closed mj-will closed 3 months ago

mj-will commented 6 months ago

In this section of the documentation, the mode-by-mode amplitude produced by PhenomHMAmpPhase is plotted but the modes argument for PhenomHMAmpPhase is not specified. Looking at the code, I don't think the default order matches what is used in the tutorial (https://github.com/mikekatz04/BBHx/blob/master/bbhx/waveforms/phenomhm.py#L122), so I think the labels are incorrect in the plot.

mikekatz04 commented 3 months ago

This is fixed. Thank you for pointing this out!