Skip to content

Enable GBFS floating vehicles by default - #4012

Merged
leonardehrenfried merged 1 commit into
opentripplanner:dev-2.xfrom
ibi-group:remove-floating-bike-feature
Mar 24, 2022
Merged

leonardehrenfried merged 1 commit into
opentripplanner:dev-2.xfrom
ibi-group:remove-floating-bike-feature

Conversation

@leonardehrenfried

Copy link
Copy Markdown
Member

Summary

The feature flag FloatingBike was introduced because linking these vehicles was buggy and the link edges were not cleaned up correctly.

This was fixed (I believe) in #3351.

For this reason we can enable this by default.

If we want to be more careful we could also just set the default to true.

Issue

none, but someone asked on the mailing list: https://groups.google.com/g/opentripplanner-users/c/cF8rKfRtrPg/m/CjvXJeJjBAAJ?utm_medium=email&utm_source=footer

Unit tests

no

Code style

yes

Documentation

yes

@leonardehrenfried
leonardehrenfried requested a review from a team as a code owner March 22, 2022 14:01
@optionsome

Copy link
Copy Markdown
Member

I've seen some station based networks that also expose the vehicles. I'm not sure if exposing them always makes sense but I'm in favour of setting this to be default on but I think we could still keep the parameter. Ideally, I think the configuration should be on updater level but it's not really a requirement for us currently and if it becomes, we can add that possibility.

@hannesj

hannesj commented Mar 22, 2022

Copy link
Copy Markdown
Contributor

I've seen some station based networks that also expose the vehicles. I'm not sure if exposing them always makes sense but I'm in favour of setting this to be default on but I think we could still keep the parameter. Ideally, I think the configuration should be on updater level but it's not really a requirement for us currently and if it becomes, we can add that possibility.

The mapFreeVehicleStatus only returns vehicles which are not part of any station, so that is already covered

@leonardehrenfried
leonardehrenfried force-pushed the remove-floating-bike-feature branch from bae02e8 to e1c4f6a Compare March 22, 2022 14:56
@leonardehrenfried

Copy link
Copy Markdown
Member Author

Spoke to @t2gran and he preferred to just enable the flag by default.

@leonardehrenfried
leonardehrenfried force-pushed the remove-floating-bike-feature branch from e1c4f6a to 0d87358 Compare March 22, 2022 14:57
@optionsome

Copy link
Copy Markdown
Member

But what if, with a network like the one in Helsinki, the user pauses the trip and leaves the vehicle somewhere (not to a station). I guess we also store and expose vehicles that are reserved?

@t2gran t2gran added this to the 2.2 milestone Mar 23, 2022
@t2gran t2gran added the !Improvement A functional improvement or micro feature label Mar 23, 2022
@leonardehrenfried
leonardehrenfried merged commit c18cc17 into opentripplanner:dev-2.x Mar 24, 2022
@leonardehrenfried
leonardehrenfried deleted the remove-floating-bike-feature branch March 24, 2022 11:34
t2gran pushed a commit that referenced this pull request Mar 24, 2022
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.

4 participants