Loop Tool Mirror Bug #71333

Closed
opened 2019-11-04 09:47:30 +01:00 by Pasang Bomjan · 8 comments

System Information
Operating system: Windows-10-10.0.18362 64 Bits
Graphics card: GeForce GT 730/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 391.35

Blender Version
Broken: version: 2.82 (sub 1), branch: master, commit date: 2019-11-03 23:35, hash: blender/blender@e45cfb574e
Worked: (optional)

Addon Information
Name: LoopTools (4, 6, 8)
Author: Bart Crouch

Short description of error
'Space' operator of the loop tool addon doesn't work with mirror modifier

Exact steps for others to reproduce the error
Open the attached blend file, select all the vertices and run the 'Space' operator

Screenshot (170).png

And this weird thing happens.
Screenshot (171).png

I have to turn the mirror modifier off to get the desired result.

It only happens when trying to space the vertices in an edge where the first and the last vertices are clipped at the mirror center [by the active mirror modifier].

I'm using blender 2.8 but this bug happens in 2.7x and 2.8 as well.

Loop Tool Bug.blend

**System Information** Operating system: Windows-10-10.0.18362 64 Bits Graphics card: GeForce GT 730/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 391.35 **Blender Version** Broken: version: 2.82 (sub 1), branch: master, commit date: 2019-11-03 23:35, hash: `blender/blender@e45cfb574e` Worked: (optional) **Addon Information** Name: LoopTools (4, 6, 8) Author: Bart Crouch **Short description of error** 'Space' operator of the loop tool addon doesn't work with mirror modifier **Exact steps for others to reproduce the error** Open the attached blend file, select all the vertices and run the 'Space' operator ![Screenshot (170).png](https://archive.blender.org/developer/F7928328/Screenshot__170_.png) And this weird thing happens. ![Screenshot (171).png](https://archive.blender.org/developer/F7928338/Screenshot__171_.png) I have to turn the mirror modifier off to get the desired result. It only happens when trying to space the vertices in an edge where the first and the last vertices are clipped at the mirror center [by the active mirror modifier]. I'm using blender 2.8 but this bug happens in 2.7x and 2.8 as well. [Loop Tool Bug.blend](https://archive.blender.org/developer/F7928347/Loop_Tool_Bug.blend)
Author

Added subscriber: @Pasang

Added subscriber: @Pasang
Member

Added subscriber: @BrendonMurphy

Added subscriber: @BrendonMurphy
Member

hi, I think this is a limitation in the addon as it's not expecting the mirror modifier.
I've tested your file and yes there's an issue. The issue is easily avoided using a better workflow.
If I remove the mirror modifier, use the "space" in looptools then add the modifier is the best way. This way your not confusing the addon.
I also tried uncheck the axis in the modifier (effectively disabling it) then use the "space" in looptools, this works too.
I also unchecked the "use in edit mode" toggle at the top of the modifier and then use the "space" in looptools, this works too.
I think in this case we could just prevent the use of the tool with all modifiers or try to disable specific modifiers in edit mode in looptools but that may just be overkill given it's very easy to avoid this issue.
I'll add something to the documentation about this limitation in the script at some point.
Leaving open for now in case anyone want's to do the modifier disable in the addon or until I get to the documentation for this addon.
Anyone wanting to do the disable modifiers in edit mode would need to run a check for active modifiers on the active object in the looptools space at the start of the execute and restore the modifier at the end of the execute. imo, this would probably cause more bugs.

hi, I think this is a limitation in the addon as it's not expecting the mirror modifier. I've tested your file and yes there's an issue. The issue is easily avoided using a better workflow. If I remove the mirror modifier, use the "space" in looptools then add the modifier is the best way. This way your not confusing the addon. I also tried uncheck the axis in the modifier (effectively disabling it) then use the "space" in looptools, this works too. I also unchecked the "use in edit mode" toggle at the top of the modifier and then use the "space" in looptools, this works too. I think in this case we could just prevent the use of the tool with all modifiers or try to disable specific modifiers in edit mode in looptools but that may just be overkill given it's very easy to avoid this issue. I'll add something to the documentation about this limitation in the script at some point. Leaving open for now in case anyone want's to do the modifier disable in the addon or until I get to the documentation for this addon. Anyone wanting to do the disable modifiers in edit mode would need to run a check for active modifiers on the active object in the looptools space at the start of the execute and restore the modifier at the end of the execute. imo, this would probably cause more bugs.
Author

Yes, I've been disabling the mirror modifier to get the desired result till now. Hope someone fixes it. There really shouldn't be any limitation..??

Yes, I've been disabling the mirror modifier to get the desired result till now. Hope someone fixes it. There really shouldn't be any limitation..??

Added subscriber: @cwolf3d

Added subscriber: @cwolf3d
Vladimir Spivak self-assigned this 2019-11-05 22:13:54 +01:00

The reason is because Loop Tool does not account for Merge vertices within the merge threshold in Mirror modifiers. If you uncheck the 'Merge', it works out correctly. I will correct this behavior, but testing will need to be done.

The reason is because Loop Tool does not account for Merge vertices within the merge threshold in Mirror modifiers. If you uncheck the 'Merge', it works out correctly. I will correct this behavior, but testing will need to be done.

This issue was referenced by ac864d9a10

This issue was referenced by ac864d9a1033024269c021d34cf5e95ecad25b94

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'
Sign in to join this conversation.
No Milestone
No project
No Assignees
4 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#71333
No description provided.