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

Sunsetting splot #188

Open
martinfleis opened this issue Jan 5, 2025 · 8 comments
Open

Sunsetting splot #188

martinfleis opened this issue Jan 5, 2025 · 8 comments

Comments

@martinfleis
Copy link
Member

martinfleis commented Jan 5, 2025

As per PySAL dev team discussions, splot is scheduled to be archived in future. Functionality for plotting of Moran statistics has been moved to esda to the module where the statistics itself is implemented. Same is happening over in giddy.

A few questions on what should happen next.

  1. What should happen with VBA plotting and related tools? There's no clear home for it anywhere else in the PySAL I think.
  2. What about convenience functions for subplots?
  1. How to approach sunsetting? Deprecation notices in splot pointing to esda? Deprecation notice on import?

cc @pysal/devs

@jGaboardi
Copy link
Member

  1. What should happen with VBA plotting and related tools?

Perhaps libpysal if nothing else fits?

  1. What about convenience functions for subplots?

Is there any drawback to including at least [1] & [4] in esda?

  1. How to approach sunsetting? Deprecation notices in splot pointing to esda? Deprecation notice on import?

I'd vote for deprecation notice on import.

@martinfleis
Copy link
Member Author

Is there any drawback to including at least [1] & [4] in esda?

Not really, just more work :).

@martinfleis
Copy link
Member Author

I just realised that there is also some stuff that uses Bokeh for plotting in _viz_bokeh.py that is not present in the API reference. Should that be moved somewhere?

@knaaptime
Copy link
Member

I'd vote for deprecation notice on import.

same

@gegen07
Copy link
Member

gegen07 commented Jan 6, 2025

  1. I believe that esda is the best fit for VBA plotting. While it's not tied to ESDA packages, exploratory spatial data analysis aligns conceptually with VBA plotting.

  2. Adding both [1] and [4] would be great enhancement for esda

  3. I agree with James and Eli.

@darribas
Copy link
Member

My two cents, in case they’re helpful in any way:

What should happen with VBA plotting and related tools? There's no clear home for it anywhere else in the PySAL I think.

My first thought was mapclassify with all functionality optional and conditional on the viz libs installed (I know there was a lot of work to keep the library pretty lightweight and all focused on classification only. But my sense would be that VBA is a way of classifying that is bi-variate.

If that doesn’t work for some reason (e.g., too much work), esda as suggested would be a fitting home too.

What about convenience functions for subplots?

My sense would be esda as most (all?) functionality falls within exploratory data analysis and, in particular, analysis of spatial autocorrelation patterns.

@sjsrey
Copy link
Member

sjsrey commented Jan 15, 2025

+1 on moving the vba classification into mapclassify

@martinfleis
Copy link
Member Author

Alright, I have opened remaining PRs for esda and am happy to finish draft in giddy as soon as I get some feedback there.

Happy with any decision regarding VBA but will probably leave that for someone else to tackle.

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

6 participants