remkos / rads

Radar Altimeter Database System (RADS)
Other
35 stars 19 forks source link

radsxogen reports wrong delta-time for Envisat #196

Closed remkos closed 4 months ago

remkos commented 4 months ago

When running radsxogen -Sn1/b the default maximum time interval should be 0.5 cycles. Instead the output reports 15.0 days (instead of 17.5 days). Though the maximum time interval used is still (correctly) 17.5 days.

This is because the later Phase C had a repeat period of 30 days. In fact, if the call would be radsxogen -Sn1 it would use correctly (by default) a time interval of 17.5 days for Phase B and 15.0 days for Phase C.

So only the output given was incorrect and confusing.

A patch is needed to report on output instead the duration either in cycles (in case of default, or when --dt is specified in cycles) or in days, with "cycles" or "days" in the output.

remkos commented 4 months ago

Fixed by commit b90398c

remkos commented 3 months ago

This fix is part of release v4.6.3 (https://github.com/remkos/rads/releases/tag/v4.6.3)