-
Notifications
You must be signed in to change notification settings - Fork 26
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
Add support for distance-between() or something similar #129
Comments
I'm in favor. I just want to propose to name it Wrt its functionality, how about this description (based on how Returns the distance in meters of either a nodeset of geopoints or a single geoshape value or a single geotrace value (not a combination of these) on Earth, in the sequence provided by the |
That sounds great to me, @MartijnR! I agree with |
And just to refresh my memory, we agreed not to namespace custom Xpath functions because XLSForms just passes them through, right? |
I had to look that up as well. We left it here. Clearly we are bad, and are in trouble if official XPath specs use the same function name (for functions that do something else). However, it would be harder to shelter users from scary namespaces if we use them for XPath functions (would require more pyxform/etc magic to do - by adding them to the XForm output). |
We have contributed the distance-between() and several other functions to ODK since last December but I can see those were never merged to the main branch: https://bitbucket.org/m.sundt/javarosa/commits/7f3677c51d8c65f3ab285c542d6924f6d77775ae I just remembered there was a public discussion with arguments whether those should be merged or not but I really don't remember what was the final result of this discussion. If I find it, I will post the link here. |
Oh, thank you, @MartijnR. I saw your last comment after I had posted my comment above. :) |
Hi I am adding the |
It it's in square meters. See https://opendatakit.github.io/xforms-spec/#fn:area |
From https://groups.google.com/d/msg/opendatakit-developers/gQrS20_LMBc/GX25qPOcAwAJ
We have support for area() and it'd be nice to get something like distance-between().
The text was updated successfully, but these errors were encountered: