Skip to content
This repository has been archived by the owner on Jun 6, 2019. It is now read-only.

Add support for additional geo data types #3

Open
bjohare opened this issue Jan 26, 2016 · 6 comments
Open

Add support for additional geo data types #3

bjohare opened this issue Jan 26, 2016 · 6 comments

Comments

@bjohare
Copy link
Contributor

bjohare commented Jan 26, 2016

Currently when collecting data from GeoODK to ONA to Cadasta only point data is accepted. It is hard-coded that way currently. Polygons and polylines should also be accepted.

@bjohare
Copy link
Contributor Author

bjohare commented Jan 26, 2016

@danielduhh don't suppose you know where I can get an example of the raw json data coming from ONA that includes Polygon and Polyline geom types?

@danielduhh
Copy link
Contributor

@bjohare I haven't seen a Polygon or Polyline type in ONA. It looks like geoshapes and geotraces are supported by xlsform (http://xlsform.org/#question-types). You could author a survey and some test responses in ODK Collect with these data types to see how they're handled in the api/v1/data/{{form_id}}

@wonderchook
Copy link

@bjohare I'm wondering if the data isn't coming out of ONA incorrectly perhaps? I can walk you through how to add that type of data. You have to use GeoODK instead of ODK to create it.

@bjohare
Copy link
Contributor Author

bjohare commented Jan 26, 2016

@wonderchook thanks, a walkthrough would be good.. @danielduhh thanks for the suggestions.

@danielduhh
Copy link
Contributor

@bjohare anytime!

@bjohare
Copy link
Contributor Author

bjohare commented Jan 26, 2016

@wonderchook don't worry.. think I have it! Will let you know if I need a few pointers..

ian-ross added a commit that referenced this issue Jan 29, 2016
Fix for #3 Add support for additional geom data types.

Looks fine!  The only thing I was wondering about was the polygon closure test (textually identical coordinate strings, i.e. with no tolerance in the testing), but that appears to be exactly what's required by the GeoJSON standard.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants