[SEDONA-738] Add moran i autocorrelation.
Did you read the Contributor Guide?
- Yes
Is this PR related to a ticket?
SEDONA-738
What changes were proposed in this PR?
MoranI index
How was this patch tested?
Unit tests
Did this PR include necessary documentation updates?
Yes
I need to add docs
and update the description of the MR
Thanks for this Pawel.
I think we need an ST Function for this. Currently it is asymmetric with the other stats functions in this regard.
I also believe that we should deprecate these function call interfaces for geostats functions and tell users to use the ST functions instead. If we go this direction I would advise against making python and public scala functions for morans I (other than ST functions for the df interfaces of course)
@james-willis Is your SQL function framework able to support Pawel's Moran'I case as well? Maybe you can point him to the right place for this?
Sure. We should be able to wrap the function in this PR with an implementation of a PhysicalFunction.
Here are the existing ones.
Here is the PR that added SQL support to geostats
@Imbruced I will merge this for now but the doc needs significant improvement. I believe we need some comprehensive end-to-end examples of each Stat function
Filed a ticket for ST function: https://github.com/apache/sedona/issues/2115