Bmesh Operator "diameter" attribute is not diameter but radius #52923

New Issue

Nikolai Janakiev · 2017-09-27T16:15:58+02:00

Nikolai Janakiev commented

2017-09-27 16:15:58 +02:00

{F888474}Blender Version
Broken: 2.79 5bd8ac9

Short description of error
Bmesh Operators (bmesh.ops.create_circle, bmesh.ops.create_uvsphere, bmesh.ops.create_icosphere, bmesh.ops.create_cone) using the attribute 'diameter' (and attributes 'diameter1' and 'diameter2' for bmesh.ops.create_cone) are creating meshes where 'diameter' is interpreted as radius of the mesh, meaning that the resulting diameter is twice as big.

Exact steps for others to reproduce the error
The script "diameter_test.py" creates a mesh with the Bmesh operators with diameter 1 but having the resulting diameter 2. In diameter_test.blend is the whole scene with the generated meshes and the previous script.

[diameter_test.py](https://archive.blender.org/developer/F888473/diameter_test.py) {[F888474](https://archive.blender.org/developer/F888474/diameter_test.blend)}**Blender Version** Broken: 2.79 5bd8ac9 **Short description of error** Bmesh Operators (bmesh.ops.create_circle, bmesh.ops.create_uvsphere, bmesh.ops.create_icosphere, bmesh.ops.create_cone) using the attribute 'diameter' (and attributes 'diameter1' and 'diameter2' for bmesh.ops.create_cone) are creating meshes where 'diameter' is interpreted as radius of the mesh, meaning that the resulting diameter is twice as big. **Exact steps for others to reproduce the error** The script "diameter_test.py" creates a mesh with the Bmesh operators with diameter 1 but having the resulting diameter 2. In diameter_test.blend is the whole scene with the generated meshes and the previous script.

Nikolai Janakiev commented

2017-09-27 16:15:58 +02:00

Changed status to: 'Open'

Nikolai Janakiev commented

2017-09-27 16:15:58 +02:00

Added subscriber: @paramcookie

blender-admin commented

2017-09-27 16:15:58 +02:00

blender/blender#52922 was marked as duplicate of this issue

Nikolai Janakiev changed title from ~~Bmesh Operator "diameter" attribute~~ to Bmesh Operator "diameter" attribute is not diameter but radius

2017-09-27 17:41:48 +02:00

Ray molenkamp commented

2017-09-28 17:39:22 +02:00

Added subscribers: @Sergey, @LazyDodo

Sergey Sharybin was assigned by Ray molenkamp

2017-09-28 17:39:22 +02:00

Ray molenkamp commented

2017-09-28 17:39:22 +02:00

I can confirm the issue, however fixing it by either renaming the parameter or changing the way the geometry gets generated in bmo_primitive.c would cause quite some breaking for everyone (which i guess we could do for 2.8). not sure what the best way forward is here, paging @Sergey for guidance.