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.

Sign up for GitHub

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

Jump to bottom

fix(core-utils): optimize flex combination generation #747

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix(core-utils): optimize flex combination generation #747

wants to merge 1 commit into from

Commits on Jun 4, 2024

  1. fix(core-utils): optimize flex combination generation 

    This accomplishes two things. First, it only allows FLEX_DIRECT on its own, without transit or anything else. Although this should work, OTP has a bug that causes issues with this configuration. It doesn't make sense to combine them anyway.

Second, it makes FLEX (access/egress) *only* comaptible with TRANSIT. As far as I can tell, OTP doesn't support FLEX combined with bikeshare or anything else. This should reduce the number of queries we send to OTP, which will improve performance.

Lastly, it improves the test output to make it easier to read.
    @daniel-heppner-ibigroup
    daniel-heppner-ibigroup committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    331a898 View commit details
    Browse the repository at this point in the history