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

Add Extended Examples #3885

Conversation

JustAnotherCodemonkey
Copy link
Contributor

@JustAnotherCodemonkey JustAnotherCodemonkey commented Jun 23, 2023

Checklist

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

Description
Add examples, especially regarding wgpu for GPGPU.

Testing
(Most of these I don't really know how to test. I guess the testing method is running it and seeing if what goes to screen / fs is the same as what was advertised.)

@JustAnotherCodemonkey JustAnotherCodemonkey marked this pull request as ready for review June 27, 2023 05:28
@JustAnotherCodemonkey JustAnotherCodemonkey marked this pull request as draft June 28, 2023 20:12
@JustAnotherCodemonkey
Copy link
Contributor Author

Re-drafting since I decided to add at least one more example for compute workgroups.

…ures to the table to accurately represent the examples.\n\nFor the new features, not all old examples may be fully represented.
…examples as well as hints on how examples build off of each other.
@cwfitzgerald cwfitzgerald changed the title Just another codemonkey/add examples Add Extended Examples Sep 14, 2023
@cwfitzgerald
Copy link
Member

@jimblandy @teoxoy @nical could someone from moz-land take a look at this, I've done a solid first pass on it and the PR is strong, but am really struggling to find the bandwidth to do a second pass.

Copy link
Contributor

@nical nical 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 a lot! A few minor adjustments before merging this. In particular let's first clarify the parts around map_async and double check that it's a reliable pattern to use in general.

examples/hello-workgroups/src/shader.wgsl Outdated Show resolved Hide resolved
examples/render-to-texture/src/shader.wgsl Outdated Show resolved Hide resolved
examples/repeated-compute/src/main.rs Outdated Show resolved Hide resolved
examples/repeated-compute/src/main.rs Outdated Show resolved Hide resolved
examples/uniform-values/src/main.rs Outdated Show resolved Hide resolved
@JustAnotherCodemonkey JustAnotherCodemonkey requested a review from a team as a code owner September 23, 2023 00:30
@JustAnotherCodemonkey
Copy link
Contributor Author

@cwfitzgerald Since I updated the README, do you think you could lift the change requested that was regarding the README?

Copy link
Contributor

@nical nical left a comment

Choose a reason for hiding this comment

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

Thanks a lot for your patience!

@cwfitzgerald
Copy link
Member

Just need conflicts fixed and we can land!

@cwfitzgerald cwfitzgerald merged commit f738551 into gfx-rs:trunk Oct 8, 2023
23 checks passed
@JustAnotherCodemonkey JustAnotherCodemonkey deleted the JustAnotherCodemonkey/add-examples branch October 8, 2023 06:07
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.

4 participants