Remnant-fits.md

@@ -12,7 +12,7 @@ The conversions are handled in by the [`pesummary.gw.file.conversions` module](h
 *  [nrutils wrappers](https://git.ligo.org/lscsoft/pesummary/-/blob/master/pesummary/gw/file/nrutils.py):
 * [x]  I only find the two functions that use `args[0]` etc a bit dangerous. (`_bbh_final_spin_precessing_using_non_precessing_fit` and `_bbh_final_spin_precessing_projected`)
 However, continuing to read in [conversions.py](https://git.ligo.org/lscsoft/pesummary/-/blob/master/pesummary/gw/file/conversions.py), it seems passing things as positional arguments is the overall design, and can't be easily fixed without global redesign, so this is ok.
-    > I have changed the code to make the arguments explicit rather than args[0]. This does not solve the problem in its entirety as this would require, as you say, a global redesign.
+    > I have changed the code to make the arguments explicit rather than args[0]. This does not solve the problem in its entirety as this would require, as you say, a global redesign. See commit: f2c694a9
 
 * [conversions.py](https://git.ligo.org/lscsoft/pesummary/-/blob/master/pesummary/gw/file/conversions.py)
 * [ ]  `final_mass_of_merger_from_NR` and `final_mass_of_merger` take `final_spin` arguments (presumably for the Healy et al fits), but don't pass them on. Since these extra arguments are only used in nrutils for minor speedup, this is probably not a problem, but it looks odd; maybe just drop them from your local interface?