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

Visual artifacts on the road #118

Open
stonier opened this issue Jun 5, 2018 · 4 comments
Open

Visual artifacts on the road #118

stonier opened this issue Jun 5, 2018 · 4 comments
Milestone

Comments

@stonier
Copy link
Collaborator

stonier commented Jun 5, 2018

Due probably to objects being co-planar - not as dire as that discussed in maliput/delphyne#150, but still some artifacts around unless you zoom in close to the road.

visual_artifacts_on_the_road

@stonier stonier added this to the Milestone 3 milestone Jun 5, 2018
@basicNew
Copy link

basicNew commented Jun 5, 2018

I think that is a bug on the road generation side, as I recall seeing the same thing on a mesh visualizer when loading the road network. @agalbachicar can you confirm that?

@agalbachicar
Copy link
Collaborator

I noticed the same thing when I restored the application some days ago. I also noticed that when changing the zoom the artifacs changed size / position. I will take a look ant try to identify the source of the problem.

@stonier
Copy link
Collaborator Author

stonier commented Sep 7, 2018

@agalbachicar did you discover anything here? I suspect the light and dark violet manifolds of the road are what they refer to as z-fighting. I've had this before, and usually took the stupidly simple shortcut of making sure the two surfaces are offset by a sufficient epsilon (but I suspect that just pushes the z-fighting problem out).

Note: googling z-fighting and opengl turns up alot of information about this problem.

@agalbachicar
Copy link
Collaborator

I did a quick FU, and the only thing I could discover that even with a z-difference when you zoom out enough the artifacts are visible again and appear erratically.

Do you think this became important enough to allocate some time to remove it?

@clalancette clalancette modified the milestones: Milestone #3, ToDo Feb 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants