Skip to content

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

Merged
Nir-Az merged 2 commits into
realsenseai:developmentfrom
Tamir91:new-3d-camera-models-for-rs-viewer
Jul 13, 2023
Merged

Nir-Az merged 2 commits into
realsenseai:developmentfrom
Tamir91:new-3d-camera-models-for-rs-viewer

Conversation

@Tamir91

@Tamir91 Tamir91 commented Jul 12, 2023

Copy link
Copy Markdown
Contributor

Tracked on [LRS-685]

D455 and D457 cameras use d455.h file
Comment thread src/gl/camera-shader.cpp Outdated
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
Copy Markdown
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
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Tested

@Nir-Az

Nir-Az commented Jul 12, 2023

Copy link
Copy Markdown
Collaborator

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

@Nir-Az

Nir-Az commented Jul 12, 2023

Copy link
Copy Markdown
Collaborator

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

@Nir-Az

Nir-Az commented Jul 13, 2023

Copy link
Copy Markdown
Collaborator

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

@Nir-Az Nir-Az left a comment

Copy link
Copy Markdown
Collaborator

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 realsenseai: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