Merge pull request #520 from nmearl/fix-issue-517

Better error message for incompatible units
astropy · Sep 26, 2019 · 60a1653 · 60a1653
2 parents 3636758 + 748ca21
commit 60a1653
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/specutils/fitting/fitmodels.py b/specutils/fitting/fitmodels.py
@@ -806,6 +806,13 @@ class instantiation.
 
                     v = current_value.to(m_orig_param_quantity.unit,
                                          equivalencies=u.equivalencies.spectral_density(dispersion))
+                else:
+                    raise ValueError(
+                        "The parameter '{}' with unit '{}' is not convertible "
+                        "to either the current flux unit '{}' or spectral "
+                        "axis unit '{}'.".format(
+                            m_orig_param.name, m_orig_param.unit,
+                            spectrum.flux.unit, spectrum.spectral_axis.unit))
 
             else:
                 v = getattr(m_in, pn).value