Merge pull request #90 from HERA-Team/pspecdata_scalar_pol_kwarg_typo

fixes minor bug in pspecdata.PSpecData.scalar
HERA-Team · May 7, 2018 · 813145b · 813145b
2 parents 19db6ce + 2c5b47f
commit 813145b
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/hera_pspec/pspecdata.py b/hera_pspec/pspecdata.py
@@ -906,7 +906,7 @@ def delays(self):
         else:
             return utils.get_delays(self.freqs[self.spw_range[0]:self.spw_range[1]]) * 1e9 # convert to ns    
 
-    def scalar(self, pol='I', taper='none', little_h=True, 
+    def scalar(self, pol, taper='none', little_h=True, 
                num_steps=2000, beam=None):
         """
         Computes the scalar function to convert a power spectrum estimate
@@ -917,10 +917,9 @@ def scalar(self, pol='I', taper='none', little_h=True,
 
         Parameters
         ----------
-        pol: str, optional
+        pol: str
                 Which polarization to compute the scalar for.
                 e.g. 'I', 'Q', 'U', 'V', 'XX', 'YY'...
-                Default: 'I'
 
         taper : str, optional
                 Whether a tapering function (e.g. Blackman-Harris) is being
@@ -1149,7 +1148,7 @@ def pspec(self, bls1, bls2, dsets, input_data_weight='identity', norm='I',
 
                 # Compute scalar to convert "telescope units" to "cosmo units"
                 if self.primary_beam is not None:
-                    scalar = self.scalar(taper=taper, little_h=True)
+                    scalar = self.scalar(p, taper=taper, little_h=True)
                 else: 
                     raise_warning("Warning: self.primary_beam is not defined, "
                                   "so pspectra are not properly normalized",