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

Fix WebGL crash due to rectangle shader returning arrays #2413

Merged
merged 1 commit into from
Jun 13, 2023

Conversation

Wumpf
Copy link
Member

@Wumpf Wumpf commented Jun 13, 2023

What

Works around gfx-rs/naga#2380

Checklist

PR Build Summary: https://build.rerun.io/pr/2413

Docs preview: https://rerun.io/preview/26e3636/docs
Examples preview: https://rerun.io/preview/26e3636/examples

@Wumpf Wumpf added 🕸️ web regarding running the viewer in a browser 💣 crash crash, deadlock/freeze, do-no-start 🦟 regression A thing that used to work in an earlier release labels Jun 13, 2023
Copy link
Member

@emilk emilk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sure you test both locally and web!

@Wumpf Wumpf merged commit 6dd8c70 into main Jun 13, 2023
@Wumpf Wumpf deleted the andreas/fix-rectangle-broken-webgl branch June 13, 2023 17:19
emilk pushed a commit that referenced this pull request Jun 15, 2023
<!--
Open the PR up as a draft until you feel it is ready for a proper
review.

Do not make PR:s from your own `main` branch, as that makes it difficult
for reviewers to add their own fixes.

Add any improvements to the branch as new commits to make it easier for
reviewers to follow the progress. All commits will be squashed to a
single commit once the PR is merged into `main`.

Make sure you mention any issues that this PR closes in the description,
as well as any other related issues.

To get an auto-generated PR description you can put "copilot:summary" or
"copilot:walkthrough" anywhere.
-->

### What

Works around  gfx-rs/naga#2380

### Checklist
* [x] I have read and agree to [Contributor
Guide](https://github.com/rerun-io/rerun/blob/main/CONTRIBUTING.md) and
the [Code of
Conduct](https://github.com/rerun-io/rerun/blob/main/CODE_OF_CONDUCT.md)


<!-- This line will get updated when the PR build summary job finishes.
-->
PR Build Summary: https://build.rerun.io/pr/2413

<!-- pr-link-docs:start -->
Docs preview: https://rerun.io/preview/26e3636/docs
Examples preview: https://rerun.io/preview/26e3636/examples
<!-- pr-link-docs:end -->
This was referenced Jun 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💣 crash crash, deadlock/freeze, do-no-start 🦟 regression A thing that used to work in an earlier release 🕸️ web regarding running the viewer in a browser
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants