Merge pull request #220 from esheldon/wmom-norm

BUG use area norm for all gauss mom things
esheldon · Jul 21, 2022 · 71e6697 · 71e6697
2 parents e656577 + d4f4b58
commit 71e6697
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/CHANGES.md b/CHANGES.md
@@ -1,5 +1,9 @@
 ## v2.1.0
 
+### bug fixes
+
+    - Fixed 1/area normalization for all moments fields in `GaussMom` results.
+
 ### new features
 
     - Added `fwhm_smooth` keyword to pre-PSF moments routines to allow for extra

diff --git a/ngmix/gaussmom.py b/ngmix/gaussmom.py
@@ -58,6 +58,13 @@ def _measure_moments(self, obs):
         res['flux'] *= fac
         res['flux_err'] *= fac
         res['pars'][5] *= fac
+        res["mom"] *= fac
+        res["mom_cov"] *= fac**2
+        res["mom_norm"] *= fac
+        res["sums"] *= fac
+        res["sums_cov"] *= fac**2
+        res["wsum"] *= fac
+        res["mom_err"] *= fac
         return res
 
     def _set_mompars(self):