Skip to content

Add support for car and scooter rental modes - #3706

Merged
hannesj merged 16 commits into
opentripplanner:dev-2.xfrom
entur:otp2_rental_modes
Nov 25, 2021
Merged

hannesj merged 16 commits into
opentripplanner:dev-2.xfrom
entur:otp2_rental_modes

Conversation

@hannesj

@hannesj hannesj commented Oct 29, 2021 •

Copy link
Copy Markdown
Contributor

Summary

Add support for specifying the rental vehicle form factor in the routing request. Previously only bike rental was supported, and using it would return itineraries with both scooter and car rentals.

Issue

Closes #3705

Unit tests

What should be tested?

Documentation

None needed?

@hannesj
hannesj requested a review from a team as a code owner October 29, 2021 16:53
@t2gran t2gran added this to the 2.1 milestone Nov 2, 2021
@t2gran t2gran added the !New Feature A functional feature targeting the end user. label Nov 4, 2021
@optionsome

Copy link
Copy Markdown
Member

Is the "fix: Map transfer leg mode correctly" commit fixing an existing problem or one created within this pull request?

@optionsome

optionsome commented Nov 4, 2021 •

Copy link
Copy Markdown
Member

There is a RoutingModes.md file that could in theory be updated here but as stated in the document, its future is a bit unclear.

@hannesj hannesj added the Entur Test This is currently being tested at Entur label Nov 8, 2021
@t2gran
t2gran requested a review from optionsome November 9, 2021 10:15

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

I tested this with oslo area osm and https://api.entur.io/mobility/v2/gbfs/limeoslo/gbfs scooter data. With the OTP client, I used /?module=planner&fromPlace=59.9193228937592%2C10.713558197021483&toPlace=59.91200869359693%2C10.77106475830078&time=3%3A50pm&date=11-09-2021&mode=SCOOTER_RENT&maxWalkDistance=4828.032&arriveBy=false&wheelchair=false&showIntermediateStops=true&debugItineraryFilter=false&locale=en query and changed between Depart/Arrive. Strangely with the Arrive option it added a second scooter leg at the end. Also the description of the SCOOTER leg is undefined (not sure if this is just some issue with this debug client) and icon is missing for scooter (not really an issue as this is just a debug client).

Also, we are not currently using scooter data in Finland with the legacy graphql API but these changes prevent finding trips using scooters if someone is (maybe @leonardehrenfried ?), as the SCOOTER type is not added to Mode enum in the schema.

@leonardehrenfried

Copy link
Copy Markdown
Member

I tested this with a city where we have scooter and it works as expected.

Screenshot from 2021-11-12 11-51-01

Although it would probably be better if we defined a colour for the scooter leg in the debug client.

@t2gran
t2gran marked this pull request as draft November 23, 2021 09:53
@hannesj

hannesj commented Nov 24, 2021

Copy link
Copy Markdown
Contributor Author

Is the "fix: Map transfer leg mode correctly" commit fixing an existing problem or one created within this pull request?

This is an existing issue, which was fixed

@hannesj

hannesj commented Nov 24, 2021

Copy link
Copy Markdown
Contributor Author

@optionsome @leonardehrenfried The issues with the debugging client, the last leg in arrive by searches and the legacy GraphQL API should be fixed now.

@hannesj
hannesj marked this pull request as ready for review November 24, 2021 12:47
@hannesj
hannesj requested a review from optionsome November 24, 2021 12:47

@optionsome optionsome 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 to me

@hannesj
hannesj merged commit 5061a1f into opentripplanner:dev-2.x Nov 25, 2021
t2gran pushed a commit that referenced this pull request Nov 25, 2021
@hannesj
hannesj deleted the otp2_rental_modes branch November 25, 2021 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Entur Test This is currently being tested at Entur !New Feature A functional feature targeting the end user.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support car and scooter rentals from GBFS feeds

4 participants