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

Ignition Marker improvements for LRAUV #52

Open
1 of 3 tasks
mabelzhang opened this issue Oct 27, 2021 · 6 comments
Open
1 of 3 tasks

Ignition Marker improvements for LRAUV #52

mabelzhang opened this issue Oct 27, 2021 · 6 comments

Comments

@mabelzhang
Copy link
Collaborator

mabelzhang commented Oct 27, 2021

Given that all of us (@arjo129 @chapulina and I) have been seeing a big need for Ignition Markers to be more usable for debugging purposes, I'm opening this ticket to keep track of specific features we need in this repo (as opposed to opening separate issues upstream, because there are so many things to make it parallel to RViz Marker. We need to focus on a few for now).

Feel free to add.

  • Per-vertex coloring for Marker arrays.
    Why: Performance for visualizing science data. See Check yaw rates instead of positions when yoyoing (retargeted #124 to main) #133. Otherwise we can only visualize a very small number of markers without significant lag, ~400 on my Dell XPS 15 9570.
    Implementation: Refer to RViz Markers implementation, which has the colors specified as a vector for vector types including POINTS, LINE_LISTS, etc. Ignition differs in that the material field is very large, and a vector of that creates a lot of overhead. Need design decision.
  • Arrows
    Why: To debug force vectors etc. I've run into this in our GSoC 2020 project with Martiño too. It's a real limitation.
    Current workaround: One way is to use POINTS and LINE_LISTS in combination. Drawback is you won't have per-vertex coloring - see bullet above.
    Implementation: Refer to RViz Markers implementation for parameters to specify sizes for different parts of the arrow.
  • (Optional bonus) Marker textures
    Why: Continuous gradient color visualization with more resemblance to typical marine science visualizations, as opposed to discrete points visualization.
    Implementation: Marker textures have recently been added to RViz2 Triangle lists support textures ros2/rviz#719
    Example usage: https://github.com/gbalke/rviz_shader_tester
@chapulina
Copy link
Contributor

@mabelzhang
Copy link
Collaborator Author

mabelzhang commented Nov 2, 2021

Does #1156 include per-vertex coloring? Seems like it does not. It seems specific to a plugin and not for generic Marker arrays. Should per-vertex coloring be a separate ticket or are we not ticketing that upstream?

@arjo129
Copy link
Member

arjo129 commented Nov 2, 2021

Also I've opened gazebosim/gz-gui#307 for some changes to GUI. Its on a backburner as I have got to fix dynamics related issues.

@chapulina
Copy link
Contributor

Does #1156 include per-vertex coloring?

Good question, I think it's necessary, otherwise it will be tough to represent different environment data values

@chapulina chapulina transferred this issue from another repository Nov 2, 2021
@chapulina
Copy link
Contributor

I looked a bit into adding arrow markers because I thought it wouldn't take long, and also because I wanted to use axis markers to visualize multiple frames of reference (in the context of #81). It turned out that we may need some deep refactoring in ign-rendering to do it, see gazebosim/gz-rendering#493. So I think the easier route may be to publish separate markers with a cylinder and a cone that combined make an arrow 😕

@caguero caguero mentioned this issue Nov 2, 2021
40 tasks
@hidmic
Copy link
Collaborator

hidmic commented Nov 10, 2022

@arjo129 am I right to say this feature request has been captured upstream?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants