search

pysal / spaghetti

SPAtial GrapHs: nETworks, Topology, & Inference
http://pysal.org/spaghetti/
BSD 3-Clause "New" or "Revised" License
260 stars 69 forks source link

Test against shapely pre-release #679

Closed jGaboardi closed 1 year ago

jGaboardi commented 1 year ago

This PR resolves #678.

martinfleis commented 1 year ago 
- package shapely-2.0b1-py37h649dd7e_0 is excluded by strict repo priority

It seems that micromamba is not happy about mixed channels. We can potentially install It from PyPI if it makes it easier. We do it in GeoPandas as well https://github.com/geopandas/geopandas/blob/403b7fa936bcae58134efbca51e10dce7a7c6639/ci/envs/39-latest-conda-forge.yaml#L39-L41

martinfleis commented 1 year ago

This actually seems to be more complicated than expected.

  1. the env still shows 1.8.5 somehow
  2. I've noticed that there is pygeos installed as well. In that case, we need to ensure that we have set and env variable USE_PYGEOS=0 to ensure shapely is actually used. (https://geopandas.org/en/stable/getting_started/install.html#using-the-optional-pygeos-dependency)
jGaboardi commented 1 year ago
  1. this is weird.
  2. I can fix this.
jGaboardi commented 1 year ago

Solved (2), but still problems for (1).

martinfleis commented 1 year ago

Can we try the conda way again and set channel-priority to flexible? https://github.com/mamba-org/provision-with-micromamba#channel-priority

Hope it does not mess the other envs...

jGaboardi commented 1 year ago

Got it 🎉