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

[RS-Viewer] 3D D455 model added #11992

Merged

Conversation

Tamir91
Copy link
Contributor

@Tamir91 Tamir91 commented Jul 12, 2023

Tracked on [LRS-685]

D455 and D457 cameras use d455.h file
@@ -164,6 +166,7 @@ namespace librealsense
if (starts_with(dev_name, "Intel RealSense D415")) index = 0;
if (starts_with(dev_name, "Intel RealSense D435")) index = 1;
if (starts_with(dev_name, "Intel RealSense L5")) index = 2;
if (starts_with(dev_name, "Intel RealSense D455") || starts_with(dev_name, "Intel RealSense D457")) index = 3;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's shorten this to :if (starts_with(dev_name, "Intel RealSense D45"))
It will catch both.

And please verify with both D455 & D457

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested

@Nir-Az
Copy link
Collaborator

Nir-Az commented Jul 12, 2023

Please add the obj filr you created before convert to h file

@Nir-Az
Copy link
Collaborator

Nir-Az commented Jul 12, 2023

Why is this PR draft? was it tested? ready for review?

@Nir-Az
Copy link
Collaborator

Nir-Az commented Jul 13, 2023

Please add the obj filr you created before convert to h file

Added

@Tamir91 Tamir91 marked this pull request as ready for review July 13, 2023 12:41
Copy link
Collaborator

@Nir-Az Nir-Az left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great

@Nir-Az Nir-Az merged commit e165ca5 into IntelRealSense:development Jul 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants