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 error in access filter when access with time-penalty exits #5735

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

t2gran
Copy link
Member

@t2gran t2gran commented Mar 7, 2024

Summary

A critical bug was introduced with the #5715. The effect of the bug was that all normal access was filtered out when an access with time-penalty existed. So, normal transit with walk access would dissapaer from the result. When setting the access mode to WALK the system would work as normal, only if an access with opening-hours exist would the walk access be filtered away.

Issue

This was discovered at Entur 2 days ago, and we have not created an issue for it.

Unit tests

✅ The unit tests are updated to cover this case.

Documentation

🟥 Not relevant

Changelog

🟥 This bug was introduced in 2.5-SNAPSHOT, so it does not exist in the last release of OTP; Hence, do not belong in the change log.

Bumping the serialization version id

🟥 The model is not changed.

@t2gran t2gran added Bug Entur Test This is currently being tested at Entur Skip Changelog labels Mar 7, 2024
@t2gran t2gran requested a review from a team as a code owner March 7, 2024 13:47
@t2gran t2gran requested a review from vpaturet March 7, 2024 13:47
@t2gran t2gran changed the title Fix access without time-penalty is removed when access with exits. Fix error in access filter when access with time-penalty exits Mar 7, 2024
@t2gran t2gran added this to the 2.5 (next release) milestone Mar 7, 2024
Copy link

codecov bot commented Mar 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.77%. Comparing base (c82b91e) to head (87b2108).
Report is 2 commits behind head on dev-2.x.

Additional details and impacted files
@@            Coverage Diff             @@
##             dev-2.x    #5735   +/-   ##
==========================================
  Coverage      67.77%   67.77%           
  Complexity     16474    16474           
==========================================
  Files           1901     1901           
  Lines          72121    72121           
  Branches        7430     7430           
==========================================
+ Hits           48880    48882    +2     
+ Misses         20727    20726    -1     
+ Partials        2514     2513    -1     

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

@leonardehrenfried leonardehrenfried self-requested a review March 7, 2024 14:17
@t2gran t2gran merged commit d51048f into opentripplanner:dev-2.x Mar 8, 2024
5 checks passed
@t2gran t2gran deleted the fix_time_penalty_filter_bug branch March 8, 2024 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Entur Test This is currently being tested at Entur Skip Changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants