react-three-renderer icon indicating copy to clipboard operation
react-three-renderer copied to clipboard

un-indexed bufferGeometry

Open lobnico opened this issue 8 years ago • 5 comments
trafficstars

Hey ! Thanks for amazing work, I ve started really recently to use it, and encountered a problem with bufferGeometry ; not setting "index" property to null will result to fatal client errors spew

<bufferGeometry position={new THREE.BufferAttribute(this._somePosAtt, 3)} normal={new THREE.BufferAttribute(this._normals, 3)} color={new THREE.BufferAttribute(this._colors, 3)} />

--> index={null} fixex evrything

three.js make explicits (attribs.index !== null), so it should be converted from undef to nul by default

lobnico avatar Jan 29 '17 12:01 lobnico

Thanks for the report, I'll fix this.

toxicFork avatar Jan 29 '17 16:01 toxicFork

If you have a simple test case it'll help too btw :)

toxicFork avatar Jan 29 '17 16:01 toxicFork

I just hit this same issue :( any chance a fix is on the way? or a workaround?

Place1 avatar Feb 14 '18 06:02 Place1

Hi @Place1 PR #208 has fixed the issue, but I did not do a release since November. I'll do one when I get a chance. The workaround is to use index={null} if it would not be set, because it would default to undefined

toxicFork avatar Feb 17 '18 09:02 toxicFork

Yeah I ended up doing that. No worries! Cheers for your time :)

On Sat, 17 Feb 2018 at 8:27 pm, Firtina Ozbalikci [email protected] wrote:

Hi @Place1 https://github.com/place1 PR #208 https://github.com/toxicFork/react-three-renderer/pull/208 has fixed the issue, but I did not do a release since November. I'll do one when I get a chance. The workaround is to use index={null} if it would not be set, because it would default to undefined

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/toxicFork/react-three-renderer/issues/146#issuecomment-366428987, or mute the thread https://github.com/notifications/unsubscribe-auth/ALMBlqrP4qYvaK10MSygfENL31yFZjCxks5tVptzgaJpZM4LwxcM .

Place1 avatar Feb 17 '18 11:02 Place1