Skip to content

Add same submode in alternative legs filter#5548

Merged
t2gran merged 5 commits into
opentripplanner:dev-2.xfrom
entur:samesubmode-alternative-legs-filter
Dec 12, 2023
Merged

Add same submode in alternative legs filter#5548
t2gran merged 5 commits into
opentripplanner:dev-2.xfrom
entur:samesubmode-alternative-legs-filter

Conversation

@eibakke

@eibakke eibakke commented Dec 1, 2023

Copy link
Copy Markdown
Contributor

Summary

This PR adds the ability to filter for same submode in the alternative legs filter.

Issue

Closes #5300

Unit tests

All existing unit tests pass. Additionally tested manually by running locally.

Documentation

This is a trivial addition to existing functionality.

@eibakke eibakke requested a review from a team as a code owner December 1, 2023 23:03
@codecov

codecov Bot commented Dec 1, 2023

Copy link
Copy Markdown

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (e27e403) 67.19% compared to head (9786035) 67.20%.
Report is 23 commits behind head on dev-2.x.

Files Patch % Lines
...routing/alternativelegs/AlternativeLegsFilter.java 33.33% 2 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             dev-2.x    #5548      +/-   ##
=============================================
+ Coverage      67.19%   67.20%   +0.01%     
- Complexity     16040    16085      +45     
=============================================
  Files           1850     1852       +2     
  Lines          70921    71002      +81     
  Branches        7410     7410              
=============================================
+ Hits           47653    47720      +67     
- Misses         20802    20813      +11     
- Partials        2466     2469       +3     

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

@t2gran t2gran 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.

Looks good, except for the missing doc.

Comment thread src/ext/java/org/opentripplanner/ext/transmodelapi/model/EnumTypes.java Outdated
@t2gran t2gran added this to the 2.5 (next release) milestone Dec 4, 2023
@t2gran t2gran added the !Improvement A functional improvement or micro feature label Dec 4, 2023
@leonardehrenfried leonardehrenfried self-requested a review December 5, 2023 10:55
…EnumTypes.java

Co-authored-by: Thomas Gran <t2gran@gmail.com>
@leonardehrenfried

Copy link
Copy Markdown
Member

Please run mvn test and commit the generated changes.

@t2gran t2gran merged commit 337276b into opentripplanner:dev-2.x Dec 12, 2023
@t2gran t2gran deleted the samesubmode-alternative-legs-filter branch December 12, 2023 09:48
t2gran pushed a commit that referenced this pull request Dec 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

!Improvement A functional improvement or micro feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

extend AlternativeLegsFilter with sameSubmode

3 participants