Skip to content

Remove filtering of flex patterns in NeTEX import - #4493

Merged
hannesj merged 2 commits into
opentripplanner:dev-2.xfrom
entur:otp2_flex_patterns
Oct 4, 2022
Merged

hannesj merged 2 commits into
opentripplanner:dev-2.xfrom
entur:otp2_flex_patterns

Conversation

@hannesj

@hannesj hannesj commented Oct 3, 2022

Copy link
Copy Markdown
Contributor

Summary

Currently we don't import JourneyPatterns with flex stops from NeTEx files. The support for adding these, as well as the corresponding change in GTFS handling was done in #3757.

@hannesj hannesj added !Improvement A functional improvement or micro feature !Technical Debt Improve code quality, no functional changes. Entur Test This is currently being tested at Entur +NeTEx This issue is related to the Netex model/import. labels Oct 3, 2022
@hannesj hannesj added this to the 2.2 milestone Oct 3, 2022
@hannesj
hannesj requested a review from t2gran October 3, 2022 11:11
@hannesj
hannesj requested a review from a team as a code owner October 3, 2022 11:11
@codecov-commenter

codecov-commenter commented Oct 3, 2022 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.60%. Comparing base (4f2325f) to head (bcf6806).
⚠️ Report is 12870 commits behind head on dev-2.x.

Files with missing lines Patch % Lines
...api/model/timetable/TimetabledPassingTimeType.java 0.00% 5 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             dev-2.x    #4493      +/-   ##
=============================================
+ Coverage      58.55%   58.60%   +0.05%     
- Complexity     11485    11517      +32     
=============================================
  Files           1515     1517       +2     
  Lines          60309    60536     +227     
  Branches        6842     6885      +43     
=============================================
+ Hits           35313    35477     +164     
- Misses         22928    22974      +46     
- Partials        2068     2085      +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@leonardehrenfried leonardehrenfried left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IBI ha been running with these flex stops in the trip times for many month and we have not noticed problems.

@hannesj
hannesj merged commit ffe9990 into opentripplanner:dev-2.x Oct 4, 2022
@hannesj
hannesj deleted the otp2_flex_patterns branch October 4, 2022 16:15
t2gran pushed a commit that referenced this pull request Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Entur Test This is currently being tested at Entur !Improvement A functional improvement or micro feature +NeTEx This issue is related to the Netex model/import. !Technical Debt Improve code quality, no functional changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants