I’m loading a model, then once the model is loaded, I’m not able to make it have smooth shading. In my OBJLoader callback I’ve tried: model.traverse(n => { if (n.material) { n.material.flatShading = false n.material…

OBJLoader returns geometries of type BufferGeometry. Calling .computeFaceNormals() on a BufferGeometry has not effect. The method only exists because of backwards compatibility. Besides, .computeVertexNormals() has no parameters. Hence, calling the method with true as first parameter has no effect. …

@prisoner849 @Mugen87 Ah, okay, there were a bunch of Google results showing n.geometry.computeVertexNormals()/n.geometry.computeVertexNormals(true), but I guess both are outdated. I just want to download OBJ files I may find on the net and use them, with no DCC tool. So how do we do it?

It does only work with an indexed BufferGeometry and if duplicate vertices are actually merged/eliminated. three.js does not provide a method that transforms a non-indexed to an indexed geometry right now. If you have an indexed geometry, you can then call BufferGeometry.computeVertexNormals(). BT…

[image] Mugen87: If you have an indexed geometry, you can then call BufferGeometry.computeVertexNormals() . So basically if I call BufferGeometry.computeVertexNormals() and don’t see a difference, then I can assume it’s not indexed? So in this case I’d have no choice but to use a DCC to corre…

[image] trusktr: So basically if I call BufferGeometry.computeVertexNormals() and don’t see a difference, then I can assume it’s not indexed? It’s best to check if geometry.index is null. If so, using Blender or a similar tool is the easiest solution. [image] trusktr: Or did the older Ge…

[image] Mugen87: var tempGeometry = new THREE.Geometry().fromBufferGeometry( geometry ); tempGeometry.mergeVertices(); tempGeometry.computeVertexNormals(); geometry = new THREE.BufferGeometry().fromGeometry( tempGeometry ); Thanks, that works great!

Here it is put into a helper function: setSmoothGeometry(modelLoadedFromOBJLoader) function setSmoothGeometry(obj) { obj.traverse(node => { if ('geometry' in node) { const tempGeometry = new THREE.Geometry().fromBufferGeometry( node.geometry ); tempGeometry.merg…

I have the same problem. I tried to use your function but it returns this error: “Uncaught TypeError: THREE.Geometry is not a constructor”. Can you help me? How can I solve it? Thanks!!

You probably use a newer release which does not include THREE.Geometry in the core anymore. Read the following topic for more information: THREE.Geometry will be removed from core with r125 Use the approach mentioned: in How to smooth an OBJ with threejs? - #12 by Laubeee

[SOLVED] How can we smooth model loaded with OBJLoader (i.e. not flat shading)?

Questions

Jashson May 4, 2021, 3:37pm 10

For v125 and later see How to smooth an OBJ with threejs? - #11 by firstlaser

Topic		Replies	Views
flatShading on BufferGeometry (or imported model) Questions normals , buffergeometry	11	3203	March 11, 2021
How to get a smooth rendering of an .obj? Questions obj-loader	1	1161	October 19, 2021
How to smooth an OBJ with threejs? Questions obj-format	15	15415	April 29, 2024
How to get smooth mesh of the .obj? Questions loaders , geometry	4	1690	August 18, 2022
Material not reacting to flatShading Questions phong-material , flatshading	4	896	March 15, 2022

[SOLVED] How can we smooth model loaded with OBJLoader (i.e. not flat shading)?

Related topics