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 ASTC feature selection in the webgl backend #3934

Merged
merged 4 commits into from Jul 17, 2023

Conversation

expenses
Copy link
Contributor

@expenses expenses commented Jul 17, 2023

Checklist

  • Run cargo clippy.
  • Run cargo clippy --target wasm32-unknown-unknown if applicable.
  • Add change to CHANGELOG.md. See simple instructions inside file.

Connections

grovesNL/glow#244
grovesNL/glow#245

Description

Currently we report both TEXTURE_COMPRESSION_ASTC and TEXTURE_COMPRESSION_ASTC_HDR if the extension is implemented, even if only SDR or HDR is enabled.

Testing

I've been using these changes on my own branch for a while.

@expenses
Copy link
Contributor Author

Looks like the Linux CI job failed for an unrelated reason.

@Wumpf
Copy link
Member

Wumpf commented Jul 17, 2023

not great, but re-running fixed that ci failure 🙉

Copy link
Member

@Wumpf Wumpf left a comment

Choose a reason for hiding this comment

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

great work, thanks!

@@ -77,6 +77,7 @@ Bottom level categories:
- Fix OpenGL/EGL backend not respecting non-sRGB texture formats in `SurfaceConfiguration`. by @liquidev in [#3817](https://github.com/gfx-rs/wgpu/pull/3817)
- Make write- and read-only marked buffers match non-readonly layouts. by @fornwall in [#3893](https://github.com/gfx-rs/wgpu/pull/3893)
- Fix leaking X11 connections. by @wez in [#3924](https://github.com/gfx-rs/wgpu/pull/3924)
- Fix ASTC feature selection in the webgl backend. by @expenses in [#3934](https://github.com/gfx-rs/wgpu/pull/3934)
Copy link
Member

Choose a reason for hiding this comment

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

this should go into a gl section, but looks like the bug fixes section is all over the place right now. can be cleaned up some point before the next release

@Wumpf Wumpf merged commit e4eb5b3 into gfx-rs:trunk Jul 17, 2023
20 checks passed
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.

None yet

2 participants