2baOrNot2ba / dreamBeam

Radio telescope beam modeling framework
https://dreambeam.readthedocs.io
ISC License
5 stars 6 forks source link

Use full integration time when calculating nrTimSamps in on_pointing_axis_tracking #9

Open David-McKenna opened 1 year ago

David-McKenna commented 1 year ago

Hey Tobia,

Not sure if this was intentional or not, but I found that on_pointing_axis_tracking was not returning the amount of time steps I expected, and ended up tracking it down to a TimeDelta division where only the number of seconds were being used to determine the time step, discarding fractions of seconds (div by 0 error for < 1 second), or minutes/hours/etc (producing far too many time steps as a result).

This is a quick PR that fixes that issue, and changes the TimeDelta.seconds variable to TimeDelta.total_seconds() for increased accuracy when determiningthe number of requested time samples.

Cheers, David