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

Error when running docker #40

Closed
NMC92 opened this issue Oct 30, 2021 · 2 comments · May be fixed by #41
Closed

Error when running docker #40

NMC92 opened this issue Oct 30, 2021 · 2 comments · May be fixed by #41

Comments

@NMC92
Copy link

NMC92 commented Oct 30, 2021

Bonjour, I hope you're well!

I have tried building and launching my own Cosmogony, because I have my own genre of this project, albeit on a smaller scale (https://mappumappu.000webhostapp.com/index.html) so it really picked my interest!
But when I try to do a full flow of the process(meaning I fetch an ...osm.pbf from Geofabrik, build a .json with the Cosmogony project and try to run Cosmogony Explorer with it) I get an error in the tests part. I have tried the Portugal-latest and Poland-latest osm.pbf files. It does, however, build the Explorer with the files already in the Cosmogony project(for example the luxembourg_filtered.osm.pbf in the cosmogony\tests\data folder), although it shows nothing besides the UI of the explorer.
I don't know what I can do to fix it so all help would be appreciated. This would be better discussed through PMs but I can't find any contacts available here. Any help would be much appreciated! Merci beaucoup!
P.S.: There are two images attached, the first with more info of the error and the second with the result of cosmogony json build. Also, if you want to reach out to me, I'll put my e-mail in my own page here. Also, the command I'm running is this:

PATH_TO_COSMOGONY_DIR=../cosmogony_data pipenv run inv -e run-local --cosmogony-file-name=cosmogony_Portugal.json --build-dockers

error_run_json_cosmogony

result_build_cosmogony

@amatissart
Copy link
Member

Hi @NMC92
As you may have seen, maintenance has been lacking on this repository for the last few months.

The tests that are running in this last step are actually sanity checks on the number of zones found in the cosmogony file. The numbers related to Portugal may be outdated, or are no longer consistent with the configuration used in Cosmogony.
Anyway, this is a bug, the explorer launch should not be blocked by such errors.

As for the absence of map in the UI, it's due to an obsolete style URL in the basemap configuration.

Both issues should be solved by #41. Could you have a try on this branch?

@NMC92
Copy link
Author

NMC92 commented Nov 1, 2021

Hello @amatissart ! I applied the patch and it's working!! Thank you very much for the help!! I'm glad I was able to contribute by indirectly pointing these two bugs.
It's true, there could be some more maintenance. I think I could try to help as much as I can, since I did struggle a bit to set things up, so maybe I could add some additional info to the little guide here, to those that would want to get their hands dirty with the cosmogony like me. Eventually look at the code as well.
Either way, merci beaucoup!! 😃

@NMC92 NMC92 closed this as completed Nov 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants