Skip to content

Conversation

@tomerashuach
Copy link
Contributor

@tomerashuach tomerashuach commented Aug 17, 2021

Updated hard-coded presets.
Added a D450 preset.
Updated the function that chooses the correct presets.

Tested by comparing the JSONs files of the new presets (in the ticket) to the exported JSONs files from the viewer.

Edit: I have added a fix for the inaccuracy of setting an int field in the SDK with a float value from the json preset.
Edit2: added two resolutions based presets to D455 as solutions for DSO-17597.

Tracked on DSO-17183.

Copy link
Collaborator

@ev-mp ev-mp left a comment

Choose a reason for hiding this comment

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

LGTM, waiting for a confirmation with IQ before merge

default_450(p);
break;
//case ds::RS455_PID:
case ds::RS435I_PID:
Copy link
Collaborator

Choose a reason for hiding this comment

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

D455

@ev-mp ev-mp marked this pull request as ready for review September 13, 2021 06:29
@ev-mp
Copy link
Collaborator

ev-mp commented Sep 19, 2021

Go over MSVC warnings

Copy link
Collaborator

@ev-mp ev-mp left a comment

Choose a reason for hiding this comment

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

Looks good

@ev-mp ev-mp merged commit 6d79d80 into IntelRealSense:development Sep 22, 2021
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