Skip to content
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

Fix IBI fare calculators #5397

Merged

Conversation

leonardehrenfried
Copy link
Member

Summary

#5284 introduced a regression which broke IBI's fare calculators.

The problem was that leg products were not transferred correctly from the intermediary fares to the eventual result, which is fixed now.

Another problem was that HSL and IBI use different APIs for representing the fares. We hope to address this in #5396

Issue

Relates to #5396

Unit tests

Lots added and adjusted.

@leonardehrenfried leonardehrenfried added Bug IBI Developed by or important for IBI Group labels Oct 5, 2023
@leonardehrenfried leonardehrenfried requested a review from a team as a code owner October 5, 2023 11:04
@codecov
Copy link

codecov bot commented Oct 5, 2023

Codecov Report

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

Comparison is base (3a1518c) 66.60% compared to head (fd232c8) 66.59%.
Report is 2 commits behind head on dev-2.x.

Additional details and impacted files
@@              Coverage Diff              @@
##             dev-2.x    #5397      +/-   ##
=============================================
- Coverage      66.60%   66.59%   -0.01%     
- Complexity     15279    15284       +5     
=============================================
  Files           1790     1790              
  Lines          69368    69388      +20     
  Branches        7306     7307       +1     
=============================================
+ Hits           46204    46212       +8     
- Misses         20688    20700      +12     
  Partials        2476     2476              
Files Coverage Δ
...ntripplanner/ext/fares/FaresToItineraryMapper.java 94.73% <100.00%> (+0.61%) ⬆️
...tripplanner/ext/fares/impl/AtlantaFareService.java 78.12% <100.00%> (+0.11%) ⬆️
...opentripplanner/ext/fares/impl/HSLFareService.java 68.26% <100.00%> (ø)
...pentripplanner/ext/fares/impl/OrcaFareService.java 76.77% <100.00%> (+0.09%) ⬆️
...org/opentripplanner/model/fare/ItineraryFares.java 88.00% <100.00%> (+1.04%) ⬆️
...tripplanner/ext/fares/impl/DefaultFareService.java 92.07% <96.00%> (+0.50%) ⬆️
...pplanner/ext/fares/impl/HSLFareServiceFactory.java 0.00% <0.00%> (ø)

... and 4 files with indirect coverage changes

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

Copy link
Contributor

@binh-dam-ibigroup binh-dam-ibigroup left a comment

Choose a reason for hiding this comment

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

Works locally, bringing the fare info back!

@leonardehrenfried leonardehrenfried merged commit bdfda72 into opentripplanner:dev-2.x Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug IBI Developed by or important for IBI Group Skip Changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants