-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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.
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
Parameters reset after disable and re-enable #1472
Comments
Hi @PhilipAtLabrador The first point that I would make is that if you are using just one camera then you do not need to set inter_cam_sync_mode. This mode is for setting up hardware sync between multiple cameras, and the '1' mode denotes that the camera is the "master" camera in the multi-camera setup whose timestamp timings the other cameras (set to '2', or "slave") try to follow. The master camera that is set to '1' sends out trigger pulses that the slave cameras that are set to '2' listen out for. Therefore, if there is only one camera then inter_cam_sync_mode can be left on its default value of '0' (no hardware sync). If the emitter needed to be kept on 'hot standby' for a fast response then you could use dynamic reconfigure during live runtime to set the laser_power value to '1' (to keep the projector ticking over but not shut off) and set to '150' (the default laser power) just before a capture, then return its value to '1' after capture when minimizing power again. Minimizing laser power would hugely reduce the depth image's detail, but if you are not capturing during the power-down phase then this may not matter. In regard to your questions 1 and 2, @doronhi will be more familiar than myself with emitter_on_off and the recent pull. |
Hi Marty @MartyG-RealSense Thanks for your response. We are indeed using multiple cameras. I just wanted to specify all of the parameters that we are setting on one of the cameras in case this ends up being an interaction of some of the parameters. Just to make sure it is clear, my question is not about the emitter_on_off feature. It is about the enable service which puts the whole camera into a standby mode that was recently merged into the development branch with this pull request: |
Thank you very much @PhilipAtLabrador for the clarification. It would be good to seek the input of the creator of the pull. Hi @doisyg Thanks again for creating the pull. :) Would you be able to assist @PhilipAtLabrador with their questions in this case about the pull please? Thanks! |
Hi, |
Thanks very much for your input @doisyg I appreciate it :) |
Thanks indeed! I believe the difference in our scenarios is that we are setting a non-default parameter for emitter_on_off and it is reliably returning to the default value after re-enable. I suspect this is a behavior of the camera firmware. Our questions remain, which parameter return to their default values after disable/re-enable, and can this behavior be changed? |
@PhilipAtLabrador On a case today on the main librealsense GitHub about camera settings resetting to defaults after a disconnection, agrunnet the Chief Technical Officer of the RealSense Group at Intel confirmed that a camera goes to default values after a power-down. You mentioned earlier putting the camera in a 'standby' state, so I wondered if this may be what you are experiencing. |
Thanks @MartyG-RealSense I had been under the impression that @doisyg 's PR was putting the camera into a standby mode where configuration was preserved. I am now understanding that it fully stops the camera and re-enabling brings it up in a default state where it must be re-configured. |
Hi @PhilipAtLabrador Do you require further assistance with this case, please? Thanks! |
I see that the the release notes for 2.2.18 say: We would love to see the parameters be persistent. I'm not sure how you typically handle issue tickets similar to this one. Let me know if you want me to close the issue, or do something else. |
Hi @PhilipAtLabrador As developers find it useful to have feedback about the level of demand for a particular feature, I will tag @doronhi the RealSense ROs wrapper developer to highlight your wish for parameter persistency when enabling / disabling sensors. Thanks! |
As parameters persistency is listed as a known issue on the 2.2.18 release notes as you said, this suggests that it is on the ROS1 development radar already in terms of awareness of the issue. So I will close this case. Thanks for the great contributions @PhilipAtLabrador |
Adding to this thread that enabling/disabling the camera seems to reset the |
Thanks very much @doisyg for your added knowledge! |
@PhilipAtLabrador , Looking at the parameter: enable_auto_exposure, I fail to reproduce the issue. Tried the set of commands: (Feel free to improve on my regexp, I gave up on the attempts to print the whole "True" and "False" words...) Regarding the emitter_on_off - I think I noticed an issue even starting it for the first time in the launch file. It seemed to me that the value is set correctly but the pattern isn't flickering the way it does when the parameter is set interactively. If you can confirm and open another issue I'll be obliged. It could be just something I did wrong here but if that is your case, it means it is not related to the "enable/disable" feature. @doisyg , same goes with the "visual_preset" parameter - I can't reproduce: Am I doing something different? Running with the following command: |
Tested also with parameter "/camera/stereo_module emitter_enabled". Value remains after cycles of disable-enable. |
Hi @PhilipAtLabrador and @doisyg Do you require further assistance with this case, please? Thanks! |
@doronhi We also saw that dynamic_reconfigure indicated that the setting was preserved across disable/enable, however the camera was not actually in the mode indicated by dynamic_reconfigure. We have implemented work arounds now to detect that the camera is not in the mode that the driver thinks it is and re-configure it. For example with the emitter_on_off mode, if we receive too many frames in a row without the meta-data indicating that the emitter has changed states, we reset this parameter. |
Thanks, @PhilipAtLabrador. Did you happen to see this phenomenon with any other control besides the emitter_on_off? |
I believe this was also happening with the intercam_sync_mode and visual_preset, but my recollection is less certain about them. intercam_sync_mode, visual_preset, and emitter_on_off are the three non-default parameters we use. We use three workarounds which appear to make them work reliably when using enable/disable.
All of these issues appear to us to be related to disabling and re-enabling the sensor as we did not see them before we started using this functionality. |
Thanks for all the info. I will try to attend again soon. |
Hi @doronhi Does this case about togglng the IR emitter on-off in ROS1 Melodic still need to be open please? Thanks! |
It is dragging and also documented as DSO-16964 but I guess until it is resolved it should remain open. |
Thanks very much @doronhi - I have added an Enhancement tag to this case as a reminder to keep it open due to being associated with a DSO. |
The source of this behavior was found to be in the firmware. A fix was introduced (DSO-17396) and I expect it to be released soon. |
Thanks for the update!
…On Mon, Aug 2, 2021 at 9:09 PM doronhi ***@***.***> wrote:
The source of this behavior was found to be in the firmware. A fix was
introduced and I expect it to be released soon.
Will keep this issue open and updated until then.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1472 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIQPLLZV7QUB6SH7OHQ5QGTT25TXXANCNFSM4TCVBW6A>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email>
.
--
Philip Case
404-210-4923
Senior Robotics Engineer
|
As a fix for the issue in this case was implemented on 3 August 2021 in DSO-17396, I will close the case. Thanks again @PhilipAtLabrador for your report! |
We are using:
D435
Firmware 5.12.8.200
https://github.com/IntelRealSense/librealsense/releases/tag/v2.39.0
2ecaf73
We are using the enable/disable function to save power when we don't need data from the D435. One of the issues we are seeing is that after re-enabling some parameters which had been set, no longer are. In particular we see this with the emitter_on_off parameter, and with the auto exposure parameters. This is not ideal for us because it takes time to switch back into emitter_on_off mode and for the exposure to re-settle.
Other settings we are using:
6Hz
Visual preset 3(high accuracy)
emitter_on_off = true
inter_cam_sync_mode = 1
Our questions are:
Thanks!
The text was updated successfully, but these errors were encountered: