-
Notifications
You must be signed in to change notification settings - Fork 40
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
front: only fetch simulation results if train_schedule_summaries is not empty #4530
Conversation
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.
Not tested but lgtm :) (and +1000 for the renaming of getTimetable, it's on my todolist)
Codecov Report
@@ Coverage Diff @@
## dev #4530 +/- ##
============================================
- Coverage 19.93% 19.93% -0.01%
Complexity 2316 2316
============================================
Files 863 863
Lines 88059 88073 +14
Branches 2271 2271
============================================
Hits 17553 17553
- Misses 69250 69264 +14
Partials 1256 1256
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
I still have the issue when importing trains from open data.
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.
Tested, the error is gone, thanks!
5a6a681
to
d388a7e
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.
Tested 👍
d388a7e
to
00b3e8c
Compare
Very good job 👍 |
closes #4521
The bug comes from the fact that we fetch the simulation results even if the timetable has a train_schedule_summaries empty