Skip to content
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

Question: Do D435's color and depth have the same clock? #8643

Closed
Egor18 opened this issue Mar 22, 2021 · 6 comments
Closed

Question: Do D435's color and depth have the same clock? #8643

Egor18 opened this issue Mar 22, 2021 · 6 comments

Comments

@Egor18
Copy link

Egor18 commented Mar 22, 2021

Required Info
Camera Model D400

Hi!
I'm wondering if rs::frame::get_timestamp() for depth and color streams uses the same clock on D435?
(Assuming we're using RS2_TIMESTAMP_DOMAIN_HARDWARE_CLOCK)
For example, if we got timestamp 1234 from depth_frame.get_timestamp() and timestamp 1234 from color_frame.get_timestamp() do they correspond to the same moment or each sensor has its own clock?

Thank you.

@MartyG-RealSense
Copy link
Collaborator

MartyG-RealSense commented Mar 22, 2021

Hi @Egor18 A good starting point for answering your question would be the comments in these links:

#2224 (comment)

#4551 (comment)

The link below expands upon some of the principles that the above two links discuss.

#1957 (comment)

@Egor18
Copy link
Author

Egor18 commented Mar 23, 2021

Thanks @MartyG-RealSense,

So I assume that there are 2 different clocks for depth and color.
But are there any guarantees on them being synched/launched at the same time?

Because here for example _camera_time_base is initialized no matter what type of frame it is:
https://github.com/IntelRealSense/realsense-ros/blob/4c52e767a1c4d0cf2fcd84bf6c577f6097c723bb/realsense2_camera/src/base_realsense_node.cpp#L1691

_camera_time_base is then used to calculate system time for every other frame, but if there are 2 completely independent clocks there should be 2 different base times.

@MartyG-RealSense
Copy link
Collaborator

MartyG-RealSense commented Mar 23, 2021

I answer cautiously as advanced metadata principles is outside of my knowledge. So I will point you towards further answers from RealSense team members with greater specialist knowledge in this particular subject.

#7016

#1548 (comment)

#2188 (comment)

Thanks for your patience in this case!

@MartyG-RealSense
Copy link
Collaborator

Hi @Egor18 Do you require further assistance with this case, please? Thanks!

@Egor18
Copy link
Author

Egor18 commented Mar 30, 2021

No, thanks.

@Egor18 Egor18 closed this as completed Mar 30, 2021
@MartyG-RealSense
Copy link
Collaborator

Thanks very much @Egor18 for the update!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants