Skip to content
This repository has been archived by the owner on Feb 3, 2025. It is now read-only.

Scene camera name clashes when using multiple CameraSensors #1811

Closed
osrf-migration opened this issue Dec 23, 2015 · 5 comments
Closed

Scene camera name clashes when using multiple CameraSensors #1811

osrf-migration opened this issue Dec 23, 2015 · 5 comments
Labels
6.0 bug Something isn't working major rendering

Comments

@osrf-migration
Copy link

Original report (archived issue) by Elte Hupkes (Bitbucket: ElteHupkes).


Say we have two identical models in the world, which are equipped with a camera sensor. Though the model names are different, the local camera sensor names will be identical (due to the models being the same internally). If we look at CameraSensor.cc:117, scene cameras are created using the local sensor name, rather than the scoped name. This means that whenever one of the models is deleted, and RemoveCamera is called in CameraSensor::173, it is going to delete the camera of the first sensor that was added, regardless of whether this is the camera corresponding to the sensor.

The camera should use the fully scoped name of the sensor instead to prevent this issue. See https://bitbucket.org/ElteHupkes/gazebo/commits/2a7c5d6cd9b3aef9015d67866030040503aa87d5 for suggested fix.

@osrf-migration
Copy link
Author

Original comment by Elte Hupkes (Bitbucket: ElteHupkes).


  • Edited issue description

@osrf-migration
Copy link
Author

Original comment by Ian Chen (Bitbucket: Ian Chen, GitHub: iche033).


good find, I'll see if I have time to write a test for this

@osrf-migration
Copy link
Author

Original comment by Ian Chen (Bitbucket: Ian Chen, GitHub: iche033).


pull request #2081

@osrf-migration
Copy link
Author

Original comment by Ian Chen (Bitbucket: Ian Chen, GitHub: iche033).


  • changed state from "new" to "resolved"

@osrf-migration
Copy link
Author

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).


  • changed state from "resolved" to "closed"

@osrf-migration osrf-migration added major rendering bug Something isn't working 6.0 labels Apr 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
6.0 bug Something isn't working major rendering
Projects
None yet
Development

No branches or pull requests

1 participant