Skip to content
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

Expose trigger for boundingbox_camera sensor #321

Closed
gsisinna opened this issue Feb 14, 2023 · 2 comments · Fixed by #322
Closed

Expose trigger for boundingbox_camera sensor #321

gsisinna opened this issue Feb 14, 2023 · 2 comments · Fixed by #322
Labels
enhancement New feature or request

Comments

@gsisinna
Copy link

Desired behavior

I would like to use a trigger to get snapshots from a boundingbox_camera sensor rather than a continuous stream.
This implementation is already present for the camera sensor, but it is not inherited by boundingbox_camera.

Alternatives considered

Decreasing the sensor update rate and setting <always_on> false </always_on> in the SDF file does not seem to work for the specific case.

Implementation suggestion

The same implementation of CameraSensor:

/// \brief True if camera is triggered by a topic

if (_sdf.CameraSensor()->Triggered())

if (this->dataPtr->isTriggeredCamera &&

this->isTriggered = true;

Additional context

Thank you!

@gsisinna gsisinna added the enhancement New feature or request label Feb 14, 2023
@mjcarroll
Copy link
Contributor

@gsisinna are you interested in implementing this? We would appreciate it!

@gsisinna
Copy link
Author

Sure @mjcarroll, I can start working on this 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants