PandABlocks / PandABlocks-server

TCP server exposing an ASCII interface to functional blocks
Apache License 2.0
2 stars 3 forks source link

added PCAP ARM timestamp #12

Closed valerixb closed 3 years ago

valerixb commented 3 years ago

Hi. I added a timestamp in the streaming header when the PCAP block is armed; this should solve issue #7 . Anyway, the header is now different from before (it has one more line), so it might not be backward-compatible.

Araneidae commented 3 years ago

Can you please also add documentation for the new arm_time field to docs/capture.rst

valerixb commented 3 years ago

OK: I made the changes; I'd prefer not to squash the first two commits because the first one has the timestamp in local time, which was requested by one of our beamlines; I agree that UTC may be a better choice, but I'd like to keep the code (which corresponds to an internal release).

valerixb commented 3 years ago

OK: done. Thank you.

Araneidae commented 3 years ago

Thank you very much, this looks good to me.