Resolve "Missing TPS deviations from TPS in the TCB timer deviations"

Closes #113 (closed)

• Rename time quantities to have very explicit names. We assume that all quantities relating one frame to another is expressed in terms of deviations (to limit the dynamical range).
  • tps_proper_time_deviations becomes tps_wrt_tcb, spacecraft proper time deviations with respect to TCB (upsampled from orbit files) [s]
  • local_timer_deviations_withinitial becomes the_wrt_tps_local, clock time deviations with respect to spacecraft proper times (contain the clock offers, drifts of various orders, and clock noise [s]
• Add time quantities
  • integrated_clock_noise_fluctuations_withinitial and integrated_clock_noise_fluctuations containing the integrated clock noise fluctuations (initially as fractional frequency deviations) [s]
  • the_wrt_tcb_withinitial, clock time deviations with respect to TCB (combinations of TPS wrt TCB and THE wrt TPS) [s]
  • moc_time_correlations, downsampled version of the_wrt_tcb_withinitial [s]
• Rename TCB synchronization noise to MOC time correlation noise
  • tcb_sync noise function is renamed moc_time_correlation
  • sync_asds parameter is renamed moc_time_correlation_asds
  • tcb_sync_noises noise time series is renamed moc_time_correlation_noises
• Use the_wrt_tps_local to compute the TDIR assistance tones
• Propagate the_wrt_tps_local to form the the_wrt_tps_distant, and then compute the MPRs
• Invert the the_wrt_tps_local to form the tps_wrt_the to resample measurements to THE

changed milestone to %v1.2

added bug label

marked this merge request as draft

added 1 commit

• c078da0e - Adjust simulation of moc time correlation so that it also works well with...

Compare with previous version

added 1 commit

• 1798f99f - Correct for pytest

Compare with previous version

It works now also in presence of initial moc_time_correlations. However, the user needs to be careful here: When creating data one has to set:

t0 => orbits_t0 + telemetry_dt * initial_telemetry_size

otherwise the times of the initial moc_time_correlations lie out of the range of the orbit file. I thought about adjusting t0 in general but decided against it, since initial moc_time_correlations are a very particular use case and it might not be justified to complicate the simulation of t0 just for that.

marked this merge request as ready

Hi @Nik! Thanks a lot for your help on this.

I've tried to check everything, commit by commit, and proposed some feedbacks. Some are minor things, others are more important (incorrect modeling). Feel free to comment on the threads!

added 1 commit

• 0ac1dc08 - Add the_wrt_tps for resampling the beatnotes to the and minor corrections

Compare with previous version