Skip to content
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

transfer of closed pr from pysal-meta #19

Open
sjsrey opened this issue Mar 25, 2019 · 1 comment
Open

transfer of closed pr from pysal-meta #19

sjsrey opened this issue Mar 25, 2019 · 1 comment

Comments

@sjsrey
Copy link
Member

sjsrey commented Mar 25, 2019

pysal/pysal#1055

@ljwolf
Copy link
Member

ljwolf commented Mar 25, 2019

We would need to sift through these more to figure out what is still needed, but my recollection is that @pedrovma and I worked pretty closely together to get all this stuff in already to the current spreg release. It appears only 2 things are missing, and only 1 of these is substantive (the spdot in the concentrated log likelihoods):

  • the init has the sur imports (upstream, here)
  • The chisqprob deprecation fix was removed (upstream, here) somehow when it came here. I've read that chisqprob was removed, but I am still able to run from scipy.stats import chisqprob on version 1.2.1 of scipy, so I'm not sure what's up. A deprecation fix for this was retained in diagnostics_sp.py, (upstream, here)
  • Changes to diagnostics_sur.py appear to be applied correctly. (upstream, here)
  • The removal of the extra regime_err_sep is done (upstream, here).
  • Changes to the concentrated log likelihood remain undone (upstream, here).
  • Regimes patch was applied. (upstream, here)
  • Spatial diagnostics for ML models is enforced. (upstream, here), as well as later changes to the SUR summary (upstream, here)
  • Changes to sur_lag.py look applied. (upstream, here), as well as to sur.py (upstream, here), sur_error.py (upstream, here) and sur_utils.py (upstream, here)

The remaining changes appear to not be things we need (e.g. referring to pysal.spreg as econometrics), and changes to testing that have been merged, or changes to documentation that we decided together not to merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants