aslam_cv2
aslam_cv2 copied to clipboard
only waiting for one complete nframe
This whole "drop incomplete nframes" logic is base on the assumption of monotonically increasing timestamps within every image topic (i.e., image messages of the same topic can not "overtake" each other). Therefore, I don't see any reason at all why one should wait for more than one complete nframe before dropping all incomplete older nframes (because given the assumption above, they can't every be completed anyways). Hence the switch from 2 to 1.
I'm not sure if that could have any side effects further down the pipeline and would prefer to not change it without adding further tests. Did you encounter any issues that would be fixed with this patch?