Skip to content

Auto removed from d500 popup, remained in d400 - #12722

Merged
Nir-Az merged 3 commits into
realsenseai:developmentfrom
remibettan:power-line-freq-d400-d500
Mar 5, 2024
Merged

Nir-Az merged 3 commits into
realsenseai:developmentfrom
remibettan:power-line-freq-d400-d500

Conversation

@remibettan

Copy link
Copy Markdown
Contributor

Tracked by: RSDEV-1777


_ds_color_common->register_color_options();

color_ep.register_option(RS2_OPTION_POWER_LINE_FREQUENCY,

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.

As I explained in the ticket.
D555e OHM is same as D400.
Meaning use a global shutter and should have AUTO value.

Only D585S should not
So looks like we will need 2 PR's

1 for dev to enable only for D555e + D400 and 1 for other repo

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.

OK - missed that line in the requirements - I am on it

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.

Done - no need for another PR I beleive

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.

Nice hack

@Nir-Az
Nir-Az merged commit 1acf84c into realsenseai:development Mar 5, 2024
@remibettan
remibettan deleted the power-line-freq-d400-d500 branch July 29, 2025 14:10
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