BoundBox/Dimensions of curve not showing correct values #90808

New Issue

Germano Cavalcante · 2021-08-20T17:24:25+02:00

2021-08-20 17:24:25 +02:00

System Information
Operating system: Windows-10-10.0.22000-SP0 64 Bits
Graphics card: Radeon (TM) RX 480 Graphics ATI Technologies Inc. 4.5.14760 Core Profile Context 20.45.37.01 27.20.14537.1001

Blender Version
Apply Scale issue:
Broken: version: 3.0.0
Caused by b9febb54a4

Undo selection issue:
Broken: version: 2.83.13, branch: master, commit date: 2021-03-09 07:21, hash: bb50046730
Worked: version: 2.80 (sub 74), branch: master, commit date: 2019-07-11 13:50, hash: 06312c6d2d

Short description of error
Ref #90736 (Curve: Apply Scale Problems with Dimensions & Shape)
When applying scale or selecting a curve and undoing it, the bound box has wrong dimensions.

Exact steps for others to reproduce the error

open attached file
select curve object
Undo selection
undo_curve_selection_bug.blend

Another file to test (from #91983):

dimensions.mp4

dimensions.blend

**System Information** Operating system: Windows-10-10.0.22000-SP0 64 Bits Graphics card: Radeon (TM) RX 480 Graphics ATI Technologies Inc. 4.5.14760 Core Profile Context 20.45.37.01 27.20.14537.1001 **Blender Version** **Apply Scale issue:** Broken: version: 3.0.0 Caused by b9febb54a4 **Undo selection issue:** Broken: version: 2.83.13, branch: master, commit date: 2021-03-09 07:21, hash: `bb50046730` Worked: version: 2.80 (sub 74), branch: master, commit date: 2019-07-11 13:50, hash: `06312c6d2d` **Short description of error** Ref #90736 (Curve: Apply Scale Problems with Dimensions & Shape) When applying scale or selecting a curve and undoing it, the bound box has wrong dimensions. **Exact steps for others to reproduce the error** - open attached file - select curve object - Undo selection [undo_curve_selection_bug.blend](https://archive.blender.org/developer/F10296109/undo_curve_selection_bug.blend) --- Another file to test (from #91983): [dimensions.mp4](https://archive.blender.org/developer/F10808782/dimensions.mp4) [dimensions.blend](https://archive.blender.org/developer/F10808787/dimensions.blend)

Germano Cavalcante commented

2021-08-20 17:24:25 +02:00

Added subscriber: @mano-wii

blender-admin commented

2021-08-20 17:24:25 +02:00

#92500 was marked as duplicate of this issue

blender-admin commented

2021-08-20 17:24:25 +02:00

#91983 was marked as duplicate of this issue

Germano Cavalcante commented

2021-08-20 17:25:03 +02:00

Changed status from 'Needs Triage' to: 'Confirmed'

Germano Cavalcante commented

2021-08-20 17:25:03 +02:00

Added subscriber: @orbisvicis

Germano Cavalcante commented

2021-09-30 18:25:46 +02:00

I believe this problem needs special attention.
For optimization, the snap system relies heavily on the BoundBox of objects.
They are used for earlier return avoiding heavy calculations and BVH creation.
But Curves often don't have the BoundBox corresponding to the visualized object.
And with b9febb54a4, it is now much more common for Curves to have a bonded evaluated Mesh.

I believe this problem needs special attention. For optimization, the snap system relies heavily on the BoundBox of objects. They are used for earlier return avoiding heavy calculations and BVH creation. But Curves often don't have the BoundBox corresponding to the visualized object. And with b9febb54a4, it is now much more common for Curves to have a bonded evaluated Mesh.

Hans Goudey commented

2021-09-30 19:20:54 +02:00

Added subscriber: @HooglyBoogly

Hans Goudey commented

2021-09-30 19:20:54 +02:00

I think the snapping system should ignore any data calculated from the DispList on an evaluated curve object now.
Any curve object with a surface will evaluate to a mesh, so the mesh can be used instead.
Snapping to wire curves directly should use the CurveEval data.

I think the snapping system should ignore any data calculated from the `DispList` on an evaluated curve object now. Any curve object with a surface will evaluate to a mesh, so the mesh can be used instead. Snapping to wire curves directly should use the `CurveEval` data.

Germano Cavalcante changed title from ~~Undo curve selection recalculates boundbox based on curve data~~ to BoundBox/Dimensions of curve not showing correct values

2021-10-06 15:40:55 +02:00

Germano Cavalcante commented

2021-10-06 15:41:25 +02:00

Added subscribers: @ditos, @PratikPB2123