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

Insert some casts for WGSL texture attribute queries #5560

Merged
merged 3 commits into from
Nov 14, 2024

Conversation

aleino-nv
Copy link
Collaborator

  • Add missing casts when querying WGSL texture attribute parameters
  • Add a new texture sampling test with no 1d array textures (WebGPU doesn't support those)

The new test is still disabled because there are some texture binding issues in Slang-RHI, still to be fixed.

This helps to address issue #4943.

@aleino-nv aleino-nv requested a review from a team as a code owner November 14, 2024 13:22
@aleino-nv
Copy link
Collaborator Author

/format

@aleino-nv aleino-nv added the pr: non-breaking PRs without breaking changes label Nov 14, 2024
@slangbot
Copy link
Contributor

🌈 Formatted, please merge the changes from this PR

aleino-nv added a commit to aleino-nv/slang that referenced this pull request Nov 14, 2024
@aleino-nv
Copy link
Collaborator Author

/format

@slangbot
Copy link
Contributor

🌈 Formatted, please merge the changes from this PR

@aleino-nv
Copy link
Collaborator Author

/format

@slangbot
Copy link
Contributor

🌈 Formatted, please merge the changes from this PR

aleino-nv added a commit to aleino-nv/slang that referenced this pull request Nov 14, 2024
There are no 1d array textures in WGSL, so
add texture-sampling-no-1d-arrays.slang based on texture-sampling.slang, but without
1d texture arrays.

This helps to address issue shader-slang#4943.
@csyonghe csyonghe merged commit f0bc464 into shader-slang:master Nov 14, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: non-breaking PRs without breaking changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants