Moved THREE.Geometry out of core.

[mrdoob]

Moved Geometry to examples/jsm/deprecated.
Moved DirectGeometry inside of Geometry and added toBufferGeometry() in Geometry.

[Mugen87]

BufferGeometry.setFromObject() and BufferGeometry.updateFromObject() also rely on Geometry and have to be removed.

[mrdoob]

The main thing left in this PR are the modifiers and I think the sanest thing to do is to convert them to use BufferGeometry...

Won't be able to work on that until Monday though.

[Mugen87]

Um, not sure how easy that is. For some modifiers, we should maybe consider if other data structures like a half edge data structure could make sense.

For simplicity, you probably want to convert calls from new BufferGeometry().fromGeometry(); to Geometry.toBufferGeometry() for now.

[Mugen87]

During my tests I have noticed one issue:

There is now Geometry.toBufferGeometry() but it only works with mesh geometries. We actually need the logic from BufferGeometry.setFromObject() so also lines and point geometries do work.

I would introduce something like Geometry.toBufferGeometryFromObject(). Sounds a bit strange but it should do the trick for these cases.

[mrdoob]

@Mugen87 🙌 🙌 🙌

[Mugen87]

This is exciting! We've worked so long so it's possible to do this step! Glad to see we've made it! 🙌

[Mugen87]

FYI: I've setup a forum topic to inform the community about this change:

https://discourse.threejs.org/t/three-geometry-will-be-removed-from-core-with-r125/22401

[GGmemeda]

Geometry deprecation ，i want to get vertices, faces, faceVertexUvs from bufferGeometry,Is there any other good way?

[Mugen87]

@GGmemeda Please use the forum for help requests. Similar questions like the following one were already answered there:

https://discourse.threejs.org/t/facevertexuvs-for-buffergeometry/23040

[mjurczyk]

@mrdoob I think that's not possible to change wiki with a PR, so a little note here:
https://github.com/mrdoob/three.js/wiki/JSON-Object-Scene-format-4#example-of-scene-with-raw-geometry-light-and-a-material-with-multiple-uv-texture-assignment
Still refers to the original Geometry at the bottom of the doc (mentioned by user 선과 on discord - https://discord.com/channels/685241246557667386/686304808994340865/806395575482056704)

Same applies to:

• https://github.com/mrdoob/three.js/wiki/JSON-Model-format-3#core
• https://github.com/mrdoob/three.js/wiki/Getting-Started (technically it works, BoxGeometry is overridden with BoxBufferGeometry now, but for clarity it could probably use -BufferGeometry 🤔 ?)

[Mugen87]

Thanks for the feedback. I've clean up the Wiki pages.