Skip to content

Seedmincomponentlengthsnapping#188

Merged
mszell merged 3 commits into
mainfrom
seedmincomponentlengthsnapping
Jun 28, 2026
Merged

Seedmincomponentlengthsnapping#188
mszell merged 3 commits into
mainfrom
seedmincomponentlengthsnapping

Conversation

@mszell

@mszell mszell commented Jun 28, 2026

Copy link
Copy Markdown
Member

Description

Snap seed points only to minimum component length of existing bike network
Move constants to new constants.py

Related Issue

Fixes #178

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 test input data prepared, but only implementeable once bike nw can be imported

  • Unit tests
  • Integration tests
  • Manual testing

Test configuration:

  • OS:

Screenshots (if applicable)

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

@mszell mszell merged commit 67f6929 into main Jun 28, 2026
6 checks passed
@mszell mszell deleted the seedmincomponentlengthsnapping branch June 28, 2026 15:35
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.

Snap seed point to existing bike network parts with minimum component length

1 participant