-
Notifications
You must be signed in to change notification settings - Fork 27
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
Adding localC()
#68
Comments
Addressing A) above: 732a6e2 adds |
Addressing B) rgeoda and PySAL esda use conditional permutation. Have commited and pushed branching code, default
@JosiahParry comments please on current |
Hey, thanks for asking! At least for the local Geary ordering, I believe we have an open ticket, I just haven't had the time to get the patch wrangled and tested! |
@JosiahParry has helpfully contributed a prototype implementation of
localC()
in #66 #67 .A) One problem being examined is how to deal with no-neighbour observations, this may be perhaps handed by using C code already in place for handing the same problem for global Geary's C, but doubly scaling the input variable.
B) A further open problem is that the prototype permutation uses all of the observations (total permutation), but we know that for local Moran's I and local G permutatons, the current observation
i
is held out of the srt (conditional permutation). This relates to https://doi.org/10.1111/gean.12304 (https://doi.org/10.31219/osf.io/ugkhp), already contributed to spdep as #58 (and also very welcome @jeffcsauer.)The text was updated successfully, but these errors were encountered: