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.

Sign up for GitHub

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

Jump to bottom

gosl implementation and gpu compute for new WebGPU #1199

Merged
merged 24 commits into from
Sep 6, 2024
Merged

gosl implementation and gpu compute for new WebGPU #1199

merged 24 commits into from
Sep 6, 2024

Conversation

rcoreilly
Copy link
Member

This implements the compute functionality in gpu package under WebGPU and updates gosl to generate wgsl code to run under webgpu.

@rcoreilly rcoreilly marked this pull request as draft September 2, 2024 18:09
@rcoreilly
Merge branch 'main' into gosl
06d386c
@rcoreilly
fastexp updated and working
1ef8314
@rcoreilly
include dosen't repeat includes
235476c
@rcoreilly
gosl misc fixes: multi-var names unpack, recursive include
0250658
@rcoreilly
gosl properly doing type inference for calling methods on sub-fields …
5c24a71 
…of fields.. and several other fixes -- now compiling all of axon except the stupid need for 0.0 etc for floats, and uint32(1) for uints -- ugh.
@rcoreilly
automatic typing of args, assignment etc to satisfy wgsl for untyped …
5f9f73b 
…literals.
@rcoreilly
update test
67e5c36
@rcoreilly
add basic example files. going to merge existing gosl first, then rem…
2af0503 
…ove vgpu, then do datafs as separate pr.
@rcoreilly
full overhaul of sltype and slrand -- much cleaner
60ff10a
@rcoreilly
slrand fully working and tested in GPU and CPU. gosl update is now co…
7eee3eb 
…mplete.
@rcoreilly
checkin good rand shader code
82bd80d
@rcoreilly
set larger buffer sizes than low baseline default, based on supported…
a3e0d8e 
… values. it is not listening to the MaxBufferSize however -- need to update our webgpu.
@rcoreilly rcoreilly marked this pull request as ready for review September 5, 2024 17:43
kkoreilly
kkoreilly requested changes Sep 6, 2024
View reviewed changes
Copy link
Member

@kkoreilly kkoreilly left a comment

Choose a reason for hiding this comment

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

Initial comments; only reviewed through gosl/process.go so far.

base/fsx/fsx.go Outdated Show resolved Hide resolved
gpu/README.md Outdated Show resolved Hide resolved
gpu/cpipeline.go Show resolved Hide resolved
gpu/device.go Show resolved Hide resolved
gpu/gosl/README.md Outdated Show resolved Hide resolved
gpu/gosl/examples/basic/main.go Show resolved Hide resolved
gpu/gosl/examples/rand/shaders/rand.wgsl Show resolved Hide resolved
gpu/gosl/extract.go Show resolved Hide resolved
gpu/gosl/examples/rand/shaders/slrand.wgsl Show resolved Hide resolved
gpu/gosl/slrand/slrand.wgsl Show resolved Hide resolved
kkoreilly
kkoreilly requested changes Sep 6, 2024
View reviewed changes
gpu/gosl/slprint/comment.go Show resolved Hide resolved
gpu/gosl/slrand/slrand.wgsl Show resolved Hide resolved
gpu/gsystem.go Outdated Show resolved Hide resolved
gpu/gsystem.go Show resolved Hide resolved
gpu/include.go Outdated Show resolved Hide resolved
math32/fastexp.go Show resolved Hide resolved
@kkoreilly kkoreilly merged commit 87067d0 into main Sep 6, 2024
1 check passed
@kkoreilly kkoreilly deleted the gosl branch September 6, 2024 05:25
@kkoreilly
Copy link
Member

kkoreilly commented Sep 6, 2024 •
edited
Loading

Note: this is an extension of #507 / #1112, which will allow for #1134.

@kkoreilly kkoreilly mentioned this pull request Sep 6, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kkoreilly kkoreilly kkoreilly approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rcoreilly @kkoreilly