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

MarkerDisplay limits you to 100 Markers #1135

Closed
mogumbo opened this issue Aug 7, 2017 · 2 comments · Fixed by #1429
Closed

MarkerDisplay limits you to 100 Markers #1135

mogumbo opened this issue Aug 7, 2017 · 2 comments · Fixed by #1429

Comments

@mogumbo
Copy link
Contributor

mogumbo commented Aug 7, 2017

As far as I can tell, you should be able to have more than 100 Markers in your scene by doing Add->Marker in RViz and increasing the Queue Size above 100. However, this appears to have no effect. I can only see 100 Markers in RViz. What does work is to hardcode a new default size for queue_size_property_ in the MarkerDisplay constructor.

@wjwwood
Copy link
Member

wjwwood commented Aug 7, 2017

If you're sending many markers at once (only time the queue size is important) then you should probably use a MarkerArray. Also when you want to scale up things past hundreds you should try to use a "list" style marker rather than lots of individual markers if you are able, see:

http://wiki.ros.org/rviz/DisplayTypes/Marker#Rendering_Complexity_Notes

Either way, the property should work as you've described (it doesn't make sense that hard coding a new value works but changing the property does not).

@wjwwood wjwwood added the bug label Aug 7, 2017
@wjwwood wjwwood added this to the untargeted milestone Aug 7, 2017
@mogumbo
Copy link
Contributor Author

mogumbo commented Aug 7, 2017

Thank you for the tips. Trying MarkerArray was on my todo list as a possible solution, but I'm not familiar with "list" style markers yet. I'll investigate those too.

@wjwwood wjwwood removed this from the untargeted milestone May 10, 2018
130s pushed a commit to 130s/rviz that referenced this issue Aug 21, 2024
…os-visualization#1135)

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
(cherry picked from commit b30838530c1aee8f6ddcbd11db258fbd24e57935)

Co-authored-by: Alejandro Hernández Cordero <alejandro@openrobotics.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants