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

Default implementation for stops creation #27

Closed
xamanu opened this issue Oct 1, 2016 · 4 comments
Closed

Default implementation for stops creation #27

xamanu opened this issue Oct 1, 2016 · 4 comments

Comments

@xamanu
Copy link
Contributor

xamanu commented Oct 1, 2016

The current stops creation is very much custom for Florianópolis. After the abstraction work happening in (#9) I'd suggest working on a default implementation for stops generation based on OSM's Public Transport Schema 2.

Managua can be a good start as the data quality is very consistent and complying the standard data structure without exceptions.

@grote
Copy link
Owner

grote commented Oct 1, 2016

It would be nice, if you could make a list where the current implementation deviates from a proper default.

@xamanu
Copy link
Contributor Author

xamanu commented Oct 1, 2016

This is very much related to #26, but is a different issue so we can have two smaller pull requests each. But probably the main discussion is going to happen in the other issue. Here it's more about stop specific stuff.

As I said in #26 I haven't started thinking too much about it. And this issue is not to say, that all of Fenix has to be different in the default implementation. We just have to look at it and decide which parts are generic and which aren't.

But for sure the name finding part for stops without a name is very particular to Fenix. The default implementation would probably assume to have bus stop names for all stops in OSM already.

I guess as we move on, more things will be figured out and can be discussed here.

@xamanu
Copy link
Contributor Author

xamanu commented Nov 12, 2016

Actually if TripsCreatorFenix logic implemented pulling Stop objects from data.stops based on the stop ids in Route.stops, then Fenix could use the currently proposed default StopsCreator implementation and get rid of the functions add_stops_to_routes() and _fill_stops(). Maybe relevant for work in #40.

@grote grote closed this as completed in ba50617 Nov 19, 2016
@grote
Copy link
Owner

grote commented Nov 19, 2016

Good idea!

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

No branches or pull requests

2 participants