Skip to content

Testfix remove edge overlaps#120

Merged
mszell merged 2 commits into
mainfrom
testfix_remove_edge_overlaps
May 19, 2026
Merged

Testfix remove edge overlaps#120
mszell merged 2 commits into
mainfrom
testfix_remove_edge_overlaps

Conversation

@mszell
Copy link
Copy Markdown
Member

@mszell mszell commented May 19, 2026

Description

Fix edge overlap removal

Related Issue

Fixes #114

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 📝 Documentation update
  • 🧹 Code refactor (no functional changes)
  • ✅ Test update

How Has This Been Tested?

  • Unit tests
  • Integration tests
  • Manual testing

Test configuration:

  • OS:

Screenshots (if applicable)

Test case
20260519_150636

Checklist

all applicable

  • My code follows the project's style guidelines
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix/feature works
  • New and existing tests pass locally

Additional Notes

The real fix was drop_duplicates, which I still do not understand how they can appear.

@mszell mszell requested a review from Manuel-Knepper May 19, 2026 13:10
@mszell mszell merged commit f3b2a17 into main May 19, 2026
6 checks passed
@mszell mszell deleted the testfix_remove_edge_overlaps branch May 19, 2026 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Edge overlaps are not completely removed

1 participant