You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The code is not doing a great job of precise timing. It determines a delay time before sending the trigger, but then executes a bunch more code.
Oh and, more importantly, that delay time is then treated relative to “current_time”, which is set after doing the phase-matching. That is going to reduce accuracy and precision, and also makes me even more uncomfortable in terms of future-proofing.
I think it would be much better to pass around absolute times, not deltas.
Chas writes:
There was a reason, three years ago, to use relative times... but it could just have been because that was of interest to me in my offline analyses?