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

Update OTP2 documentation, add elevation data #178

Merged
merged 3 commits into from
Feb 19, 2024

Conversation

leonardehrenfried
Copy link
Contributor

This PR updates the documentation about how OTP is built and deployed. I've tried to be careful about not removing useful information and only remove those things that I'm certain are no longer up-to-date.

It also adds elevation data to the OTP graph.

@leonardehrenfried
Copy link
Contributor Author

cc @dulvui

@leonardehrenfried
Copy link
Contributor Author

As of today, I also added GBFS rental systems.

@leonardehrenfried
Copy link
Contributor Author

@dulvui Is there a deployment of OTP2 somewhere I can take a look?

@dulvui
Copy link
Contributor

dulvui commented Feb 19, 2024

@leonardehrenfried Great work thank you! Sorry for the late reply.
The test deployment with the custom frontend is running here https://journey.opendatahub.testingmachine.eu
Only the otp service with its standard frontend is running here https://otp.opendatahub.testingmachine.eu

I deployed a first version a few weeks ago, but I will update it now with your latest changes

@dulvui dulvui merged commit b463b14 into noi-techpark:main Feb 19, 2024
1 check passed
@rcavaliere
Copy link
Member

@dulvui @leonardehrenfried I think there is still something to fix, isn't it? If I test the front-end applications, I don't see public transport data retrieved.

@leonardehrenfried
Copy link
Contributor Author

So when I use the OTP debug UI I get results: http://tinyurl.com/27zobf7u

But you're right, in your UI coded against OTP1 I see no results. This is because it is using the deprecated REST API: https://docs.opentripplanner.org/en/dev-2.x/apis/Apis/

I recommend that you don't use this API anymore and migrate to one of the GraphQL APIs. The latest version of digitransit and otp-react-redux use it.

Lastly, I believe you might be able to temporarily make it work by removing the parameter bannedRoutes from the URL that the UI sends to OTP. At least this is how I did it: http://tinyurl.com/2co6jr3l

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants