Skip to content
This repository has been archived by the owner on May 27, 2024. It is now read-only.

Sort osm stops by distance from gtfs stop #45

Closed

Conversation

nereocystis
Copy link
Contributor

This patch sorts osm stops by distance from gtfs stop

For each osm stop, use TreeSet to sort all gtfs stops which are
potential matches
The sort is by distance from the osm stop

Also, clean up ReportViewer so that HashMap key and values are
declared, removing the need for casts.

close: CUTR-at-USF#44
@nereocystis
Copy link
Contributor Author

And the comments from the git checkin

sort osm stops by distance from gtfs stops

For each osm stop, use TreeSet to sort all gtfs stops which are
potential matches
The sort is by distance from the osm stop

Also, clean up ReportViewer so that HashMap key and values are
declared, removing the need for casts.

close: #44

@barbeau
Copy link
Member

barbeau commented Oct 30, 2018

@reubot Any thoughts on this PR?

@nereocystis
Copy link
Contributor Author

If there is interest, I can work on resolving the conflicts.

@reubot
Copy link
Contributor

reubot commented Nov 5, 2018

If there is interest, I can work on resolving the conflicts.

Yes, that would be good.

@reubot
Copy link
Contributor

reubot commented Feb 5, 2019

@nereocystis Are you still interested in resolving the conflicts or should I go ahead and do it?

@nereocystis
Copy link
Contributor Author

nereocystis commented Feb 7, 2019 via email

@reubot
Copy link
Contributor

reubot commented Apr 14, 2019

@nereocystis I ended up doing the merges of all three pulls requests in my pull-testing branch. https://github.com/reubot/gtfs-osm-sync/tree/pull-testing
Have a look if you want and tell me if you see any problems.

@barbeau
Copy link
Member

barbeau commented Oct 2, 2019

Replaced by #51

@barbeau barbeau closed this Oct 2, 2019
@barbeau
Copy link
Member

barbeau commented Oct 2, 2019

Thanks @nereocystis!

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

Successfully merging this pull request may close these issues.

3 participants