fix(face): Avoid removing the last vertex twice

ladybug-tools / ladybug-geometry

🐞 📦 A library with geometry objects used throughout the Ladybug Tools core libraries

https://www.ladybug.tools/ladybug-geometry/docs/

GNU Affero General Public License v3.0

23 stars 22 forks source link

fix(face): Avoid removing the last vertex twice #317

Closed chriswmackey closed 3 years ago

chriswmackey commented 3 years ago

It seems that the remove_colinear_vertices method would remove the last vertex twice if those last two vertices were equivalent to one another. This is not necessarily what you want unless both of those last two vertices are colinear with the vertices surrounding it. This commit fixes this case.

github-actions[bot] commented 3 years ago

:tada: This PR is included in version 1.23.16 :tada:

The release is available on:

GitHub release
v1.23.16

Your semantic-release bot :package::rocket: