Skip to content
This repository has been archived by the owner on Mar 1, 2021. It is now read-only.

Feed with JSON #72

Closed
karussell opened this issue Sep 29, 2016 · 3 comments
Closed

Feed with JSON #72

karussell opened this issue Sep 29, 2016 · 3 comments

Comments

@karussell
Copy link
Member

karussell commented Sep 29, 2016

Currently we accept GPX files, but JSON file (GeoJSON) would be nice too. We could reuse our work from graphhopper/graphhopper#781

@kodonnell
Copy link
Contributor

Possibly change this to creating a generic interface, with GPX and JSON as certain implementations? I suggest this as our data doesn't suit GPX (or a likely JSON format), which means I'm going to have to hack the source instead of extending it, which is a tad messy.

@karussell
Copy link
Member Author

karussell commented Nov 7, 2016

For now I would prefer to have just one implementation and convert incoming data depending on the format. But you are right: instead of having everything like the entry objects tuned and named for GPX we should aim for something more generic ("measurement"?). I'm not sure what the benefits would be if we would make e.g. the "measurement" an interface.

BTW: Ideally we could consume the same format the the web API of the core can return (type=json|gpx). And for JSON we currently not even return the time for every point but this would be important to have. Related to graphhopper/graphhopper#311

@karussell
Copy link
Member Author

Duplicate of #59

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

No branches or pull requests

2 participants