-
Notifications
You must be signed in to change notification settings - Fork 258
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
Should not update the number of elements too #211
Comments
Yea sorry, that's intended. I should probably update the docs.
There's no guarantee you're going to update all of the buffers or that they
will be the same lengths so it seemed best not to guess.
2022年10月7日(金) 3:17 Ronan Sandford ***@***.***>:
… Hi,
Thanks for twgl.js nice library!
I was playing with it and realised that updating the buffer data using
twgl.setAttribInfoBufferFromArray did not automatically update the
numElements value
As I started with a zero element array in init, it never display anything
until I updated the bufferInfo.numElements value
Is that intended ?
Is there any elegant way to update it while calling
twgl.setAttribInfoBufferFromArray
For better context, see my repoduction repo with the commit fixing my
issue : ***@***.***
<bug-reproduction/twgl-test@d2f4d36>
—
Reply to this email directly, view it on GitHub
<#211>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABZKNEWSXP2YIX6UA7FXNDWB72EFANCNFSM6AAAAAAQ7ONEDI>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
Thanks for twgl.js nice library!
I was playing with it and realised that updating the buffer data using
twgl.setAttribInfoBufferFromArray
did not automatically update the numElements valueAs I started with a zero element array in init, it never display anything until I updated the
bufferInfo.numElements
valueIs that intended ?
Is there any elegant way to update it while calling
twgl.setAttribInfoBufferFromArray
For better context, see my repoduction repo with the commit fixing my issue : bug-reproduction/twgl-test@d2f4d36
The text was updated successfully, but these errors were encountered: