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

Enable Mesa workaround also on Mesa 21 #1598

Merged
merged 1 commit into from
Mar 25, 2021

Conversation

peci1
Copy link
Contributor

@peci1 peci1 commented Mar 19, 2021

Description

This enables the Mesa workaround for Mesa 21.x, too (and all future Mesa 2?.x versions).

My system reports:

GL_RENDERER   = AMD RENOIR (DRM 3.40.0, 5.10.24-051024-generic, LLVM 11.0.1)
GL_VERSION    = 4.6 (Compatibility Profile) Mesa 21.0.0 - kisak-mesa PPA
GL_VENDOR     = AMD

The fix merged in #1588 is limited to Mesa 20.x only, but the problems it works around are present in 21.x, too.

Checklist

  • If you are addressing rendering issues, please provide:
    • Images of both, broken and fixed renderings.
    • Source code to reproduce the issue, e.g. a YAML or rosbag file with a MarkerArray msg.
  • If you are changing GUI, please include screenshots showing how things looked before and after.
  • Choose the proper target branch: latest release branch, for non-ABI-breaking changes, future release branch otherwise.
    Due to the lack of active maintainers, we cannot provide support for older release branches anymore.
  • Did you change how RViz works? Added new functionality? Do not forget to update the tutorials and/or documentation on the ROS wiki
  • While waiting for someone to review your request, please consider reviewing another open pull request to support the maintainers of RViz. Refer to the RViz Wiki for reviewing guidelines.

Signed-off-by: Martin Pecka <peckama2@fel.cvut.cz>
@rhaschke rhaschke merged commit 03fe615 into ros-visualization:melodic-devel Mar 25, 2021
@rhaschke
Copy link
Contributor

Thanks.

@peci1 peci1 mentioned this pull request Feb 15, 2022
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

Successfully merging this pull request may close these issues.

2 participants