Skip to content

impr: Make clouds use the texture.write() API and move from rgba8unorm to r8unorm since only one channel was used#2157

Merged
reczkok merged 1 commit intoreleasefrom
impr/clouds-fix
Feb 12, 2026
Merged

impr: Make clouds use the texture.write() API and move from rgba8unorm to r8unorm since only one channel was used#2157
reczkok merged 1 commit intoreleasefrom
impr/clouds-fix

Conversation

@reczkok
Copy link
Contributor

@reczkok reczkok commented Feb 11, 2026

Found it randomly while searching for for usages.

.createTexture({
size: [NOISE_TEXTURE_SIZE, NOISE_TEXTURE_SIZE],
format: 'rgba8unorm',
format: 'r8unorm',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think we only ever sample .x but please verify

@reczkok reczkok marked this pull request as ready for review February 11, 2026 13:38
@github-actions
Copy link

pkg.pr.new

packages
Ready to be installed by your favorite package manager ⬇️

https://pkg.pr.new/software-mansion/TypeGPU/typegpu@dd06d12cab9d53212277b400af33ee9401bd3cc9
https://pkg.pr.new/software-mansion/TypeGPU/@typegpu/noise@dd06d12cab9d53212277b400af33ee9401bd3cc9
https://pkg.pr.new/software-mansion/TypeGPU/unplugin-typegpu@dd06d12cab9d53212277b400af33ee9401bd3cc9

benchmark
view benchmark

commit
view commit

Copy link
Contributor

@aleksanderkatan aleksanderkatan left a comment

Choose a reason for hiding this comment

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

yee

Copy link
Collaborator

@cieplypolar cieplypolar left a comment

Choose a reason for hiding this comment

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

nice catch ☁️ ☁️ ☁️

@reczkok reczkok merged commit e806f76 into release Feb 12, 2026
3 checks passed
@reczkok reczkok deleted the impr/clouds-fix branch February 12, 2026 14:34
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.

3 participants