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

fix order in public transport analysis #1274

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

nlehuby
Copy link
Member

@nlehuby nlehuby commented May 28, 2021

put back the offset (to make sure the projection on the the ways is ok even if the same way is used multiple times in the relation)

Fix #1262

put back the offset (to make sure the projection on the the ways is ok even if the same way is used multiple times in the relation)

Fix osm-fr#1262
@frodrigo
Copy link
Member

Cannot be back as is. Was removed because of massive performances issues.

Eg try on http://download.openstreetmap.fr/extracts/south-america/brazil/southeast/rio-de-janeiro.osm.pbf

@nlehuby
Copy link
Member Author

nlehuby commented May 29, 2021

time docker-compose --project-name brazil_rio_de_janeiro run --rm backend ./osmose_run.py --country=brazil_rio_de_janeiro --skip-download --no-clean --analyser osmosis_relation_public_transport
gives me

real	7m54,332s
user	0m0,710s
sys	0m0,142s

which seems reasonable 🤔

@frodrigo
Copy link
Member

It is probably linked to postgis/postgres or under lib version.

Look at this run 260/261 was killed after hours of timeout: https://buildbot.osmose.openstreetmap.fr/#/builders/371

I was able to reproduce the issue on osm110 debian with postgres 11.10, postgis 2.5.

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.

The stops may not be in the right order 1260/11
2 participants