search

pysal / segregation

Segregation Measurement, Inferential Statistics, and Decomposition Analysis
https://pysal.org/segregation/
BSD 3-Clause "New" or "Revised" License
111 stars 26 forks source link

compute_segregation_profile is not working #191

Closed weikang9009 closed 2 years ago

weikang9009 commented 2 years ago

Function compute_segregation_profile in spatial_indexes.py gives an error as it calls the deprecated MultiInformationTheory function which is not imported in the module. Will this function be kept and moved to another module after spatial_indexes.py is removed? It is not currently listed in the API doc page of segregation.

knaaptime commented 2 years ago

thanks for raising this. Sorry for the confusion

the compute_segregation_profile function has moved to the dynamics module, and the MultiInformationTheoryIndex was renamed to MultiInfoTheory and moved to the multigroup module (I notice the docstring for compute_multiscalar_profile still hasn't been updated and suggests you pass the nonexistent MultiInformationTheory class--so that needs to be fixed).

If you import the function from dynamics, you should be able to pass any index you like, including MultiInfoTheory, like the updated demo but please let me know if that doesn't work for you.

Id like to go ahead and remove spatial_indexes.py asap to help avoid this confusion, so ill probably cut another release this month

weikang9009 commented 2 years ago

Sounds good! Thanks for the clarification!