-
Notifications
You must be signed in to change notification settings - Fork 46
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
WebGLAnimation #105
base: main
Are you sure you want to change the base?
WebGLAnimation #105
Conversation
get updates
…holder classes for `WebGLRenderingContext` & `WebGL2RenderingContext` which will become the wrappers for our native WebGLContext that is part of the native `window` class.
We can skip BoxGeometry and just use BoxBufferGeometry for now.
That was too easy. WebGLRenderer will map from the material to the actual GLSL shader.
…iona tests to check clearing memory
…type automagically.
Updating Animation branch with latest.
😃 |
|
||
// functions or properties required for testing | ||
|
||
requestAnimationFrame(animationLoop: WebGLAnimationFrame): void { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ASWebGLue
in fact now provides WebGLRenderingContext
(we could add WebGLRenderingContext
at some point, but for now I just want to get things running).
For requestAnimationFrame
, see https://github.com/trusktr/ecmassembly.
* @param gl version 1 of webgl used by most browsers | ||
* @param gl2 version 2 of webgl which supports additional texture features and custom XR and VR devices | ||
*/ | ||
setContext(gl: WebGLRenderingContext | null, gl2: WebGL2RenderingContext | null): void { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Discriminated union. :)
@ZoeDreams Gonna circle back to this and get it in. We will replace |
No description provided.