Box3#applyMatrix4()
that only supports affine transforms.geometry.applyMatrix()
- The geometry is transformed with applyMatrix()
.r93 Box3#applyMatrix4()
& r92 Box3#applyMatrix4()
- A Box3
instance is constructed from the geometry. A non-affine matrix is then applied.For BoxGeometry
, it is expected is that both operations result in equivalent Box3
instances.
The r92
version transformed each Box3
vertex, then used setFromPoints()
to resize to fit.
See https://github.com/mrdoob/three.js/issues/6039 for a similar issue with a Vector3#applyMatrix4()
optimization.
@razh Thank you for providing a well-written and clear demo.
I support reverting #13993.
/cc @mikialex
I haven't consider the case that matrix is not affined, for consistency it should be revert
Most helpful comment
@razh Thank you for providing a well-written and clear demo.
I support reverting #13993.