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

ENH: ensure lag_spatial is compatible with both W and Graph #765

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

martinfleis
Copy link
Member

Ensures that when downstream packages (like splot pysal/splot#184) use lag_spatial when a Graph is given, the actual computation produces correct lag. This will avoid silent introduction of bugs when using Graph as scipy does not raise but produces different output.

Copy link

codecov bot commented Aug 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.4%. Comparing base (d52d684) to head (4b9970d).
Report is 3 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #765   +/-   ##
=====================================
  Coverage   85.4%   85.4%           
=====================================
  Files        150     150           
  Lines      15989   15989           
=====================================
  Hits       13655   13655           
  Misses      2334    2334           
Files with missing lines Coverage Δ
libpysal/weights/spatial_lag.py 92.9% <100.0%> (ø)

@knaaptime knaaptime merged commit 514917f into pysal:main Aug 29, 2024
12 checks passed
@knaaptime
Copy link
Member

i think i would support doing a bugfix release with this change

@martinfleis martinfleis deleted the lag-compat branch August 29, 2024 17:48
@martinfleis
Copy link
Member Author

Yeah, we could do one with this and #760 but we'll need to backport those commits. Do you want me to do that?

@knaaptime
Copy link
Member

im game

@martinfleis
Copy link
Member Author

im game

had to google what that means 🙃

@knaaptime
Copy link
Member

😬 sorry. i vote we do it

@martinfleis
Copy link
Member Author

The release notes action didn't get what I was doing but the patch release is out https://github.com/pysal/libpysal/releases/tag/v4.12.1

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

Successfully merging this pull request may close these issues.

4 participants