LoopTools Circle operator does not work with the mirror modifier #74834

Closed
opened 2020-03-17 05:48:10 +01:00 by Jesse Yurkovich · 15 comments

System Information
Operating system: Windows-10-10.0.18362-SP0 64 Bits
Graphics card: GeForce GTX 1070 with Max-Q Design/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 441.66

Blender Version
Broken: version: 2.83 (sub 8), branch: master, commit date: 2020-03-14 00:31, hash: blender/blender@acab745078
Broken: version: 2.82a is also broken
Worked: 2.81a works correctly!

Addon Information
Name: LoopTools (4, 7, 2)
Author: Bart Crouch, Vladimir Spivak (cwolf3d)

Short description of error
Trying to run Circle on verts that span across the mirror line of a mirror modifier fails.

Exact steps for others to reproduce the error
bad_mirror.blend

  • Open the attached .blend (will open in 2.82a as well)
  • Run the Loop Tools -> Circle operator
  • Observe nonsense

Before operator runs:
before.png

After running Circle:
after.png

**System Information** Operating system: Windows-10-10.0.18362-SP0 64 Bits Graphics card: GeForce GTX 1070 with Max-Q Design/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 441.66 **Blender Version** Broken: version: 2.83 (sub 8), branch: master, commit date: 2020-03-14 00:31, hash: `blender/blender@acab745078` Broken: version: 2.82a is also broken Worked: 2.81a works correctly! **Addon Information** Name: LoopTools (4, 7, 2) Author: Bart Crouch, Vladimir Spivak (cwolf3d) **Short description of error** Trying to run Circle on verts that span across the mirror line of a mirror modifier fails. **Exact steps for others to reproduce the error** [bad_mirror.blend](https://archive.blender.org/developer/F8411507/bad_mirror.blend) - Open the attached .blend (will open in 2.82a as well) - Run the Loop Tools -> Circle operator - Observe nonsense Before operator runs: ![before.png](https://archive.blender.org/developer/F8411502/before.png) After running Circle: ![after.png](https://archive.blender.org/developer/F8411504/after.png)
Author
Member

Added subscriber: @deadpin

Added subscriber: @deadpin

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

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

#75173 was marked as duplicate of this issue

#75173 was marked as duplicate of this issue
Vladimir Spivak self-assigned this 2020-03-17 08:37:06 +01:00

Added subscriber: @israelsr

Added subscriber: @israelsr

I have the same problem

I have the same problem

Added subscriber: @mano-wii

Added subscriber: @mano-wii

I don't see this as a bug, but it is because this tool, like all other tools used in editing meshes, only works on the vertices/edges/faces of the original mesh.
Supporting this has many implications:

  • What if the modifier is hidden?
  • How to deal with other modifiers?

Supporting this only for the mirror modifier would be very specific and in my view it is more of a feature request than a bug.
The result in the image is not "nonsence", it is a "circle" formed with the 4 selected vertices.

I don't see this as a bug, but it is because this tool, like all other tools used in editing meshes, only works on the vertices/edges/faces of the original mesh. Supporting this has many implications: - What if the modifier is hidden? - How to deal with other modifiers? Supporting this only for the mirror modifier would be very specific and in my view it is more of a feature request than a bug. The result in the image is not "nonsence", it is a "circle" formed with the 4 selected vertices.
Author
Member

There's explicit code in the addon to handle modifiers and it worked in prior versions, as well as 2.79 (so I'll invoke the magical "regression" word to keep the bug alive :))

There's explicit code in the addon to handle modifiers and it worked in prior versions, as well as 2.79 (so I'll invoke the magical "regression" word to keep the bug alive :))

Perhaps the problem is that .to_mesh () returns the mesh without using the mirror modifier. Therefore, this modifier is not taken into account when calculating the circle. Need more research.

Perhaps the problem is that .to_mesh () returns the mesh without using the mirror modifier. Therefore, this modifier is not taken into account when calculating the circle. Need more research.
Member

Added subscriber: @Blast

Added subscriber: @Blast
Member

Added subscriber: @ankitm

Added subscriber: @ankitm
Member

line 548: "mod.use_mirror_merge = False" commenting out this seems to fix it

see merged task

> line 548: "mod.use_mirror_merge = False" commenting out this seems to fix it see merged task

This issue was referenced by af6e605a12

This issue was referenced by af6e605a120ba58b0e5822b060bedffe590fcd2b

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

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

Added subscriber: @OliverVillar

Added subscriber: @OliverVillar
Sign in to join this conversation.
No Milestone
No project
No Assignees
6 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender-addons#74834
No description provided.