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

GLSL 1.0 to 3.0 #12250

Closed
fernandojsg opened this issue Sep 21, 2017 · 8 comments
Closed

GLSL 1.0 to 3.0 #12250

fernandojsg opened this issue Sep 21, 2017 · 8 comments

Comments

@fernandojsg
Copy link
Collaborator

fernandojsg commented Sep 21, 2017

I'm doing some testing with GLSL 3.0 and three.js and I would like to know your opinion about how to deal with the future WebGL2 renderer with regards to the current WebGL1 shaders.
I've been thinking in the possible approaches:

Probably the second option is the low hanging fruit here, and it's actually the one I'm following on my experiments, as it won't need much code to modify and it should works just out of the box. Things could get more complicated as long as we'll start using specific WebGL2 features like for Uniform Buffer Objects.

Does anyone has an strong opinion here why we should choose one or another?

@mrdoob
Copy link
Owner

mrdoob commented Sep 21, 2017

I think I vote for the third approach.

@takahirox
Copy link
Collaborator

I prefer 3 (if it works perfectly).

@tentone
Copy link
Contributor

tentone commented Sep 23, 2017

+1 For the third approach!

@pailhead
Copy link
Contributor

vote yes on 3 ^

@Spaxe
Copy link

Spaxe commented Sep 26, 2017

voting for 3rd. Keeping WebGL2Renderer completely separate also means no switches between webgl 1 and 2.

@yoshikiohshima
Copy link
Contributor

I was not aware of this thread (Thanks for letting me know @takahirox ) but I just made a pull request to support WebGL2 and GLSL3.00. My patch does not address the issue of migrating existing user shader code out there, but as far as shader code in the code base, it provides full backward compatibility (with "low-tech": namely ifdefs) and you can start using new features if you want to.

@Mugen87
Copy link
Collaborator

Mugen87 commented Sep 8, 2018

Since #13717 is merged...can we close this issue?

@fernandojsg
Copy link
Collaborator Author

Sure!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants