Skip to content

[RFC] WebGL #79

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

Closed
wants to merge 3 commits into from
Closed

[RFC] WebGL #79

wants to merge 3 commits into from

Conversation

asny
Copy link

@asny asny commented May 1, 2019

Continuing from issue #5

@asny asny mentioned this pull request May 1, 2019
@richard-uk1
Copy link
Contributor

Awesome work. I'm not really placed to offer an opinion on the proposal as a whole, but I have a couple of questions:

  1. If there are some combinations of parameters that are not allowed, can we create types that represent the allowed subset using sum and product types (enums and structs)?
  2. Do we want to support WebGL rather than WebGPU, since people using wasm are going to have an up-to-date browser, and WebGPU is designed to replace WebGL?

@Pauan
Copy link
Contributor

Pauan commented Nov 15, 2019

@derekdreery I think the problem is that WebGPU isn't even remotely close to ready (it's not standardized yet, the design is still in flux, it currently only runs on Mac OS X Chrome Canary, and you have to enable an experimental flag, and it's missing a ton of features, and it's going to be buggy).

But yeah, in the long term we'll want to replace WebGL with WebGPU.

@ranile
Copy link
Collaborator

ranile commented Jun 25, 2021

Hey @ansy, can you create an issue for this RFC. See #136 for details

@asny
Copy link
Author

asny commented Jun 25, 2021

Hi @hamza1311 , it is already there #5

@ranile
Copy link
Collaborator

ranile commented Jun 25, 2021

Hi @hamza1311 , it is already there #5

You mentioned that it's a continuation. I haven't read both so can you confirm that it's all the information is there?

@ranile
Copy link
Collaborator

ranile commented Jun 25, 2021

Just read it, seems like the issue does have the information. Closing this in favor of #5

@ranile ranile closed this Jun 25, 2021
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