Three.js: RaytracingWorker: Add support for BufferGeometry

Created on 30 Jul 2018  路  5Comments  路  Source: mrdoob/three.js

I think that would be a valuable addition.

Enhancement
Source
picture WestLangley
👍1

Most helpful comment

I'll try digging into it a little

picture zz85 on 14 Aug 2018
❤2

All 5 comments

Agreed.

mrdoob picture mrdoob on 31 Jul 2018

/ping @zz85 Any interest?

WestLangley picture WestLangley on 8 Aug 2018

I'll try digging into it a little

zz85 picture zz85 on 14 Aug 2018
❤2

@zz85 I think it would be OK in this case to remove support for legacy Geometry. At some point it will not be renderable anyway.

WestLangley picture WestLangley on 14 Aug 2018

Just some updates from initial investigation - it seems like BufferGeometry has already some support in RaytracingWorker. MeshBasicMaterial works but not MeshPhongMaterial.

The reason is that checkBufferGeometryIntersection() in Mesh is not returning vertexNormals. could try modifying that method a little to see if that works.

zz85 picture zz85 on 15 Aug 2018
❤1
Was this page helpful?
0 / 5 - 0 ratings

Related issues

jack-jun picture jack-jun  路  3Comments
akshaysrin picture akshaysrin  路  3Comments
fuzihaofzh picture fuzihaofzh  路  3Comments
alexprut picture alexprut  路  3Comments
yqrashawn picture yqrashawn  路  3Comments