James Gaboardi

Results 209 issues of James Gaboardi

@ljwolf @sjsrey @weikang9009 Should all PySAL federated packages have more than 1 owner in [`pypi`](https://pypi.org)? It seems that most packages at least have multiple maintainers listed but still only have...

Discussion

Since the refactoring the notebook links in `README` are not working.

docs
maintenance

We should update the `master` branch to the `main` branch to conform with the push for reducing problematic language.

package maintenance

xref https://github.com/pysal/momepy/pull/427#pullrequestreview-1127124471 > I am thinking we should do a patch release that changes our requirements to avoid ppl running into that bug... @martinfleis Should we go ahead and pin...

Following #415/#420, we should keep an eye on progress/changes with the `flake8` functionality in `pyproject.toml` files. Currently, `flake8` is not configurable within [`pyproject.toml`](pyproject.toml) files, which leads to the need for...

admin/maint

We need to update the docs following @erinrolson's addition of the LSCP-B model (#259). * [ ] add to [Tutorials](https://pysal.org/spopt/tutorials.html) list * [ ] add to [API](https://pysal.org/spopt/api.html) docs * [...

docs
locate
GSoC2022

This PR resolves the [`PendingDeprecationWarning`](https://github.com/geopandas/geopandas/actions/runs/3243624358/jobs/5318620857#step:6:4291) that is being thrown by `matplotlib` due to the upcoming deprecation of [`matplotlib.cm`](https://matplotlib.org/stable/api/cm_api.html#module-matplotlib.cm). ``` PendingDeprecationWarning: The get_cmap function will be deprecated in a future version....

#720 introduced a [small `ruff` regression](https://results.pre-commit.ci/run/github/88305306/1683569187.sU0ZcZd7TJCHuDQRO36_Eg) where the `geom_col` keyword is no longer used in [`util._arcs_as_gdf()`](https://github.com/pysal/spaghetti/blob/main/spaghetti/util.py#L746), which triggers a `ruff` failure in pre-commit. The thought here is that maybe we...

rough edge

Following up from #369, it would be nice to implement @martinfleis' idea of throwing a warning when the CRS is geographic. Here is where that happens in [`geopandas`](https://github.com/geopandas/geopandas/blob/cf2d2a9e9cc41deb6eb93acd22a12bf66798e338/geopandas/array.py#L341-L352). What we...

enhancement
rough edge

In [`remove_false_nodes`](https://github.com/pysal/momepy/blob/main/momepy/preprocessing.py#L193), line attributes are lost when merged together. These can be critical (or at least helpful) for further analysis following preprocessing and users should have the option for retaining...

rough edge