pysal / mgwr

Multiscale Geographically Weighted Regression (MGWR)
https://mgwr.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
368 stars 126 forks source link

CI: make CI run #128

Closed martinfleis closed 1 year ago

martinfleis commented 1 year ago

Minor update trying to make CI run again. Now it fails on install.

knaaptime commented 1 year ago

ive done that copy/paste a million times. Do we want to standardize this convention? Personally i prefer the hidden dir with the . prefix

martinfleis commented 1 year ago

For the location of CI envs?

jGaboardi commented 1 year ago

Do we want to standardize this convention?

Very much yes! I prefer non-hidden, but don't have a strong opinion on it (for once :P).

martinfleis commented 1 year ago

I'd also prefer non-hidden but equally don't care.

knaaptime commented 1 year ago

it's always felt more like an auxiliary directory to me, only useful for the cloud-based workflows, not the 'user', so it feels natural to hide it just like the .github dir to clean up the structure a bit

But i also dont feel very strongly about it--and im already outvoted, so just fine to change

jGaboardi commented 1 year ago

Let's one of use make an issue for this in the submodule template to not forget. Then we can get started standardizing the repos.

martinfleis commented 1 year ago

only useful for the cloud-based workflows

I'm using those to setup my dev envs because these are more likely up to date than some dev env file created 4 years ago :).