remove pygeos

[martinfleis]

Not sure why pygeos is in requirements and CI ends here as it doesn't seem to be used in the code. Removing.

[martinfleis]

Huh? How come this PR caused a failure? Anyone closer to segregation code can tell me what is wrong?

[knaaptime]

weird. I'll dig into it

thanks for putting this together

[jGaboardi]

Very strange indeed. Seems to be coming from deep within joblib/futures?

[jGaboardi]

How and why removing the pygeos dependency would lead to that path is a bit of mystery...

[martinfleis]

It is not caused by this PR. See main https://github.com/pysal/segregation/actions/runs/4642225334/jobs/8215907349. For some reason, the workflow is marked as successful even when it failed in all envs :D.

[martinfleis]

segregation/.github/workflows/unittests.yml

Line 16 in b3643f7

continue-on-error: true

[knaaptime]

i thought the continue on error was supposed to just keep running tests on all platforms even when one fails (to make sure its not isolated to windows or something) but maybe im wrong

the good news is it looks like the issue is limited to that single inference test (based on individual-level permutation) so i'll take a closer look at that function today

[knaaptime]

(im fine to go ahead and merge this PR then and work on the failure separately)

[jGaboardi]

Looks like this started failing 5 days ago.

[martinfleis]

i thought the continue on error was supposed to just keep running tests on all platforms even when one fails

That is fail-fast.