-
Notifications
You must be signed in to change notification settings - Fork 165
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use ReadQueue for PhotonCamera timestamps #1316
base: master
Are you sure you want to change the base?
Conversation
37e3f47
to
a455ac2
Compare
What's the overhead of enabling poll storage? If it's somewhat decent we should make this a configurable option when actually creating the camera. |
It just makes the circular buffer of std vectors of bytes larger. Since the contents of the vector are dynamically allocated, the cost is zero, and then as you push more new packets in, you allocate new memory. So probably on the order of 20kb, at worst, of dynamically allocated memory, and 20*sizeof(vector) statically? |
This removes the extra GetLastChange call to keep everything properly atomic.
Closes #1303