Setting origin to Center of Mass on zero-volume objects produces wrong results now in 2.79 #52434

New Issue

Kai Kostack · 2017-08-18T09:38:55+02:00

Kai Kostack commented

2017-08-18 09:38:55 +02:00

System Information
Windows 7 x64

Blender Version
Broken: 2.79 RC
Worked: 2.78c

Short description of error
Setting the origin to "Center of Mass" for mesh objects which are not closed and manifold now produces wrong and inconsistent results. Results even change on repetition.

Exact steps for others to reproduce the error

Create a plane
Set 3D cursor to any location but (0, 0, 0)
Set origin of plane to cursor
Set origin of plane to "Center of Mass" (change the type in the operator panel)
As a result the origin won't be set to the actual center of the plane. Repeating the last step will change the result even.

The .blend only requires step 4:
median-center-on-zero-volume-wrong.blend

Martin Felke found this bug, thanks to him.

**System Information** Windows 7 x64 **Blender Version** Broken: 2.79 RC Worked: 2.78c **Short description of error** Setting the origin to "Center of Mass" for mesh objects which are not closed and manifold now produces wrong and inconsistent results. Results even change on repetition. **Exact steps for others to reproduce the error** 1. Create a plane 2. Set 3D cursor to any location but (0, 0, 0) 3. Set origin of plane to cursor 4. Set origin of plane to "Center of Mass" (change the type in the operator panel) As a result the origin won't be set to the actual center of the plane. Repeating the last step will change the result even. The .blend only requires step 4: [median-center-on-zero-volume-wrong.blend](https://archive.blender.org/developer/F717094/median-center-on-zero-volume-wrong.blend) Martin Felke found this bug, thanks to him.

Kai Kostack commented

2017-08-18 09:38:55 +02:00

Changed status to: 'Open'

Kai Kostack commented

2017-08-18 09:38:55 +02:00

Added subscriber: @KaiKostack-3

Campbell Barton was assigned by Aaron Carlisle

2017-08-20 06:41:34 +02:00

David Morris commented

2017-08-20 11:54:07 +02:00

Added subscriber: @pixelgrapher

David Morris commented

2017-08-20 11:54:07 +02:00

In 2.79 RC2, it appears that repeating step 4 causes the origin to asymptotically move closer towards the center.

JT Nelson commented

2017-08-20 16:21:50 +02:00

Added subscriber: @jta

blender-admin commented

2017-08-21 07:06:07 +02:00

This issue was referenced by 361c7cbbc5

This issue was referenced by 361c7cbbc57720db3b04698c7e45fe72297e2b24

Campbell Barton commented

2017-08-21 07:15:09 +02:00

Changed status from 'Open' to: 'Resolved'

Campbell Barton closed this issue

2017-08-21 07:15:09 +02:00

Sergej Reich commented

2017-09-05 16:50:52 +02:00

Added subscriber: @sreich

Sergej Reich commented

2017-09-05 16:50:53 +02:00

So I really don't like this.
Setting the origin already has too many options, when is "Origin to Geometry" ever actually useful, especially since we have the "Center of Mass" option.
If anything we should remove the "Geometry" option.
This is not a trivial matter as well, in the past I had several users being confused by this.

If the new method is actually better for solid objects (I would like to believe it since it's the same shoelace algorithm working on tetrahedra but haven't seen the evidence for it) then we should make it work instead of adding another option.
IMO a proper fix would be to figure out why it fails and then either fix it or detect the failing cases and use the appropriate algorithm.

So I really don't like this. Setting the origin already has too many options, when is "Origin to Geometry" ever actually useful, especially since we have the "Center of Mass" option. If anything we should remove the "Geometry" option. This is not a trivial matter as well, in the past I had several users being confused by this. If the new method is actually better for solid objects (I would like to believe it since it's the same shoelace algorithm working on tetrahedra but haven't seen the evidence for it) then we should make it work instead of adding another option. IMO a proper fix would be to figure out why it fails and then either fix it or detect the failing cases and use the appropriate algorithm.

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Setting origin to Center of Mass on zero-volume objects produces wrong results now in 2.79 #52434