Skip to content

Rename D500 temperatures options + add projector temp - #12397

Merged
Nir-Az merged 2 commits into
realsenseai:developmentfrom
Tamir91:projector-temperature
Nov 14, 2023
Merged

Rename D500 temperatures options + add projector temp#12397
Nir-Az merged 2 commits into
realsenseai:developmentfrom
Tamir91:projector-temperature

Conversation

@Tamir91

@Tamir91 Tamir91 commented Nov 12, 2023

Copy link
Copy Markdown
Contributor

Tracked on [RSDEV-1022]

@Tamir91
Tamir91 requested a review from Nir-Az November 13, 2023 07:12
Comment thread src/ds/d500/d500-device.cpp Outdated
@@ -629,12 +629,16 @@ namespace librealsense

depth_sensor.register_option(RS2_OPTION_ASIC_TEMPERATURE,

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.

This needs to change, right?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oh, I changed it...

Comment thread src/to-string.cpp Outdated
arr[RS2_OPTION_DEPTH_AUTO_EXPOSURE_MODE] = "Auto Exposure Mode";
CASE( LEFT_IR_TEMPERATURE )
CASE( OHM_TEMPERATURE )
CASE(SOC_PVT_TEMPERATURE)

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.

Align please

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Spaces added

@Tamir91
Tamir91 requested a review from Nir-Az November 13, 2023 09:51
@Nir-Az Nir-Az changed the title Options for librealsense Rename D500 temperatures options + add projector temp Nov 13, 2023
@Nir-Az
Nir-Az merged commit a312ffd into realsenseai:development Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants