-
Notifications
You must be signed in to change notification settings - Fork 44
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adapt route calculation #7475
Adapt route calculation #7475
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## dev #7475 +/- ##
============================================
- Coverage 29.36% 24.81% -4.55%
============================================
Files 1177 863 -314
Lines 145975 132784 -13191
Branches 2871 961 -1910
============================================
- Hits 42867 32953 -9914
+ Misses 101423 98877 -2546
+ Partials 1685 954 -731
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
e9042f2
to
b376162
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks !
b376162
to
1af7050
Compare
Description
close https://github.com/osrd-project/osrd-confidential/issues/496
Allow implicit nodes when they have one group.
Note
We still have a nonambiguous description of the route.
We don't need a migration since it's optional to provide implicit node positions.
Motivation
We can now split tracks in the editor. This creates a new "track section link" and breaks a lot of routes doing so. We could: