Skip to content

Draft: Proposal for md delay refactoring

Jan Kienerequested to merge
proposal_for_md_delay_refactoring into 2122_ref_delay_alignment_split_rendering_and_external_rend_crashes_omasa

The md synch delay in the renderer is read as float from the command line, but is internally quantized to 5ms (subframe length for TD renderer). My proposal is to

  • only allow integer values on the command line (it says "delay in ms" there and any thing fractional is anyway removed by the rounding)
  • do the rounding already when reading the value -> this makes internally things a bit easier when going to fixed-point later
  • bonus: fix a bug in adding the overall delay which I randomly found when discussing things with @tamarapu

If this is fine for all the renderer experts, this should be ported back to float as well.

Merge request reports