-
Notifications
You must be signed in to change notification settings - Fork 150
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
GeoJSON #66
Comments
Great idea. I'd live to have that implemented. What functions do you think should return geojson? |
I think it makes sense for the following functions:
Maybe the name could be something like funcNameAsGeoJson? |
Sounds good. Or, instead we could extend those functions to take an What do you think?
|
Ok! I agree, Let's try it that way. 2016-11-12 13:27 GMT-03:00 Brendan Nee notifications@github.com:
|
Hello, I'm kind of new at this so bear with me. I was also interested in an option to get an output in geoJSON. I added a geoJSON parameter which can be true or false (or anything really since I check only if it is there) which transform the result in a I am not really satisfied on how I introduced the new parameter, since it is optional and there are other optional parameters I had to take into account that it can take the place of the other optional parameter. You already had some code for this case but only if there is only one optional parameter. It works but feels hacky. I would also maybe pass an Anyways, this is what I did so far. I'll start working on other functions like routes and maybe also try to tackle streams, this would be a nice addition though and I've been wanting to try them for a while. |
I like your idea of passing an |
I added some new routes to node-gtfs that are geoJSON specific. This ended up working better than adding options to the existing methods. I added routes for getting stops and shapes from GTFS. I also published a new package called gtfs-to-geojson that uses these routes in a handy command line tool to generate geoJSON and save it to a file. https://github.com/brendannee/gtfs-to-geojson Check it out and let me know what you think. |
New routes might be the better idea here to keep backward compatibility. The options object idea would be better off for a new version, as it is now it creates more problem than it solves. Thanks by the way, this is nice work. I'm learning as I fiddle with your code :) |
I plan to implement versions of some functions to get GeoJSON
What do you think about it?
The text was updated successfully, but these errors were encountered: