realtimeradio / vla-dev

Ethernetization/Channelization Firmware for the VLA-COSMIC FPGA system
GNU General Public License v3.0
1 stars 3 forks source link

Remove unecessary delay tracking debugging states #43

Open talonmyburgh opened 6 months ago

talonmyburgh commented 6 months ago

At present, we have delay tracking states:

The above varying states necessitate extra code and branching which can cause performance impacts to the repeating service operating on the F-Engine nodes. There is also a required function call that could be removed. With us requiring the delay tracking service to loop at ~10Hz, removing some of this logic could improve timing in this service.

demorest commented 6 months ago

Hi Talon, I don't think we have a need for any of the "half" setups (where different things are done to the two tunings) in normal operations. Those were entirely for debugging purposes. We might occasionally want to run an observation with either nothing applied at all, or possibly with fixed delays applied but no delay tracking. Are those options currently available?