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

Activate SkyAlps, fix graph build #177

Merged
merged 7 commits into from
Jan 31, 2024
Merged

Conversation

leonardehrenfried
Copy link
Contributor

@leonardehrenfried leonardehrenfried commented Jan 29, 2024

I ran the previously deployed container image with

podman run -it -p 8080:8080 ghcr.io/noi-techpark/odh-mentor-otp/odh-mentor-otp:df521c4357bd0f7d72e18253a3af9a3cf9ce7887

and I can say that the workflow generally works, however there is no OSM data in the container. This is because osmium-tool was not installed. Therefore I'm fixing the graph build for all Github Actions by installing the dependencies in build-graph.sh.

Since the SklAlps feed is fixed, it's also re-activated.

@leonardehrenfried leonardehrenfried force-pushed the otp2 branch 4 times, most recently from d2b6c3f to 40ba071 Compare January 29, 2024 15:56
@leonardehrenfried leonardehrenfried changed the title Activate SkyAlps, fix graph build Activate SkyAlps, add elevation data, fix graph build Jan 29, 2024
@leonardehrenfried leonardehrenfried force-pushed the otp2 branch 2 times, most recently from 9a299d3 to cdb3759 Compare January 29, 2024 16:31
@leonardehrenfried leonardehrenfried changed the title Activate SkyAlps, add elevation data, fix graph build Activate SkyAlps, fix graph build Jan 29, 2024
@leonardehrenfried leonardehrenfried marked this pull request as ready for review January 29, 2024 16:56
@leonardehrenfried
Copy link
Contributor Author

cc @dulvui

@dulvui
Copy link
Contributor

dulvui commented Jan 31, 2024

Great thank you! I'll merge it now

@dulvui dulvui merged commit df65ce3 into noi-techpark:main Jan 31, 2024
1 check passed
@leonardehrenfried leonardehrenfried deleted the otp2 branch January 31, 2024 08:24
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.

2 participants