bug in bridge function in looptools #34801

Closed
opened 2013-03-28 14:58:38 +01:00 by Fred Feucht · 6 comments

Project: Blender Extensions
Tracker: Py Scripts Release
Blender: 2.66
Script name: mesh_looptools.py
Wiki page: http://wiki.blender.org/index.php/Extensions:2.6/Py/Scripts/Modeling/LoopTools
Author(s): Bart Crouch
Category: Mesh
Status: Open

%%%The bridge function in loop_tools is producing incorrect results on the second and subsequent uses. The problem occurs when the remove faces check box is selected (the default). The first time through it works fine but the next time you use it the wrong faces are deleted and also the wrong faces are selected at completion.

I've debugged this myself and the problem results from the re-use of previously deleted faces in the bm.faces list. New faces are not always added to the end of the list and this causes access of the face list by index to return incorrect results. The functions bridge_select_new_faces and bridge_remove_internal_faces need to be updated produce the correct results.%%%

**Project**: Blender Extensions **Tracker**: Py Scripts Release **Blender**: 2.66 **Script name**: mesh_looptools.py **Wiki page**: http://wiki.blender.org/index.php/Extensions:2.6/Py/Scripts/Modeling/LoopTools **Author(s)**: Bart Crouch **Category**: Mesh **Status**: Open %%%The bridge function in loop_tools is producing incorrect results on the second and subsequent uses. The problem occurs when the remove faces check box is selected (the default). The first time through it works fine but the next time you use it the wrong faces are deleted and also the wrong faces are selected at completion. I've debugged this myself and the problem results from the re-use of previously deleted faces in the bm.faces list. New faces are not always added to the end of the list and this causes access of the face list by index to return incorrect results. The functions bridge_select_new_faces and bridge_remove_internal_faces need to be updated produce the correct results.%%%
Author

Changed status to: 'Open'

Changed status to: 'Open'
Bart Crouch was assigned by Brecht Van Lommel 2013-11-18 10:03:02 +01:00

Added subscriber: @brecht

Added subscriber: @brecht
Member

Fixed in local development version. Will commit in the upcoming week.

Fixed in local development version. Will commit in the upcoming week.

This issue was referenced by 4b7f9c6819

This issue was referenced by 4b7f9c6819318b772767dad19621bbbaf5827177
Member

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'
Member

Closed by commit 4b7f9c6819.

Closed by commit 4b7f9c6819.
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#34801
No description provided.