Skip to content

Add a new Emitter frequency option #10953

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 14 commits into from
Oct 19, 2022
Merged

Add a new Emitter frequency option #10953

merged 14 commits into from
Oct 19, 2022

Conversation

@Nir-Az
Copy link
Collaborator

@Nir-Az Nir-Az commented Oct 2, 2022 •
edited
Loading

D455 & D457 devices only

  1. Add Emitter frequency new XU option
  2. Add a new Unit Test checking the option requirements (default values + not setting while streaming)
  3. Extract option_model from the model-views BIG file
  4. Refactor options drawing function for a more readable code
    Tracked on [LRS-516]

@Nir-Az Nir-Az force-pushed the pwm_freq_option branch 2 times, most recently from 10ea59e to 01ca2fe Compare October 3, 2022 11:27
@Nir-Az Nir-Az closed this Oct 11, 2022
@Nir-Az Nir-Az reopened this Oct 11, 2022
@Nir-Az Nir-Az requested a review from remibettan October 11, 2022 18:44
remibettan
remibettan reviewed Oct 18, 2022
View reviewed changes
Copy link
Contributor

@remibettan remibettan left a comment

Choose a reason for hiding this comment

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

Few comments - thanks!

src/ds5/ds5-options.h
public:
explicit emitter_frequency( uvc_sensor & ep,
const std::map< float, std::string > & description_per_value );
void set( float value ) override;
Copy link
Contributor

Choose a reason for hiding this comment

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

Needed? See corresponding comment in the cpp file

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Same answer

Copy link
Contributor

Choose a reason for hiding this comment

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

ok

unit-tests/live/d400/test-emitter-frequency.py Outdated
################################################################################################

test.start("Verify camera defaults")
device_nane = device.get_info(rs.camera_info.name)
Copy link
Contributor

Choose a reason for hiding this comment

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

typo naMe

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Will rename thanks

Copy link
Contributor

@remibettan remibettan Oct 19, 2022 •
edited
Loading

Choose a reason for hiding this comment

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

Has it been corrected?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes

Copy link
Contributor

Choose a reason for hiding this comment

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

ok

@Nir-Az Nir-Az requested a review from remibettan October 19, 2022 11:34
remibettan
remibettan reviewed Oct 19, 2022
View reviewed changes
Copy link
Contributor

@remibettan remibettan left a comment

Choose a reason for hiding this comment

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

fewer comments...

remibettan
remibettan approved these changes Oct 19, 2022
View reviewed changes
Copy link
Contributor

@remibettan remibettan left a comment

Choose a reason for hiding this comment

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

LGTM

@Nir-Az Nir-Az merged commit e45b1fa into IntelRealSense:D4XX_MIPI Oct 19, 2022
@lupak
Copy link

lupak commented Apr 4, 2023

Hi! I've updated my camera firmware version to 5.14.0 and the realsense version to v2.53.1. How do I access and set this setting?

@Nir-Az
Copy link
Collaborator Author

Nir-Az commented Apr 4, 2023 •
edited
Loading

Hi! I've updated my camera firmware version to 5.14.0 and the realsense version to v2.53.1. How do I access and set this setting?

Can you share which camera model are you using?

@lupak
Copy link

lupak commented Apr 4, 2023

D430

@lupak
Copy link

lupak commented Apr 4, 2023

Ahh I might have missed reading the top part of the 1st comment. Is it for D455 & D457 devices only?
If yes, is there a way to replicate it for the D430s?
We have a situation where the laser is affecting wireless mic systems in classrooms.

@Nir-Az
Copy link
Collaborator Author

Nir-Az commented Apr 5, 2023

I just added it so it will be clearer 😀.
I will check about your question and update.

@lupak
Copy link

lupak commented Apr 5, 2023

Thank you!

@Nir-Az
Copy link
Collaborator Author

Nir-Az commented Apr 16, 2023

@lupak I can to update you that we are discussing this issue internally and I believe we will add this feature implementation into or FW + SDK, but it will take some time because laser related implementation and validation require some extra processes.
Please monitor future releases for this feature.
Thanks

@lupak
Copy link

lupak commented Apr 18, 2023

Thanks for the update @Nir-Az

@lupak
Copy link

lupak commented Sep 17, 2024

Hi! Is there any update on this being available on the D430 cameras?

@Nir-Az
Copy link
Collaborator Author

Nir-Az commented Sep 18, 2024

@lupak Hi,
Looks like this internal threat didn't reach to a conclusion,
Let me recheck that and get back with an answer..

@Nir-Az Nir-Az deleted the pwm_freq_option branch September 30, 2024 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@remibettan remibettan remibettan approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Nir-Az @lupak @remibettan