Skip to content

Commit

Permalink
change the default theta_inc_sampling to 8, to account for the effect…
Browse files Browse the repository at this point in the history
… of the incidence by default.
  • Loading branch information
ghislainp committed Aug 12, 2024
1 parent a7b7757 commit fc91295
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions smrt/rtsolver/nadir_lrm_altimetry.py
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ class NadirLRMAltimetry(object):
_broadcast_capability = {} # "theta_inc", "polarization_inc", "theta", "phi", "polarization"}

def __init__(self, waveform_model=None, oversampling=10, return_oversampled=False, skip_pfs_convolution=False,
return_contributions=False, theta_inc_sampling=1, return_theta_inc_sampling=False, error_handling="exception"):
return_contributions=False, theta_inc_sampling=8, return_theta_inc_sampling=False, error_handling="exception"):
# """

# """
Expand Down Expand Up @@ -178,7 +178,7 @@ def convolve_with_PFS_PTR_PDF(self, t_gate, backscatter, t_inc_sample):
# we've no PFS, we need numerical integration
pfs = self.PFS_numerical(t_gate)

# now take into account the ptr + pdf
# now take into account the PTR + PDF assuming Gaussian PTR and PDF
if (self.sensor.pulse_sigma > 0) or (sigma_surface > 0):
gaussian_norm = 0.3989422804014328
sigma_c = np.sqrt(self.sensor.pulse_sigma**2 + (2 * sigma_surface / C_SPEED)**2)
Expand Down

0 comments on commit fc91295

Please sign in to comment.