photonvision icon indicating copy to clipboard operation
photonvision copied to clipboard

Add sequence ID, capture, publish and recieve timestamp to PhotonPipelineResult

Open mcm001 opened this issue 2 years ago • 3 comments

Closes #1304

mcm001 avatar Mar 27 '24 21:03 mcm001

Would this be a good time to make that change that includes the pipeline name in the result. Not sure if that overhead is worth.

srimanachanta avatar Apr 01 '24 14:04 srimanachanta

Maybe just the index number is a good compromise?

mcm001 avatar Apr 01 '24 14:04 mcm001

We also sent across the capture timestamp in system time for future syncing.

lgtm otherwise.

TheTripleV avatar Apr 24 '24 03:04 TheTripleV