pysal / mgwr

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

Joblib update to default -1 #138

Closed Ziqi-Li closed 11 months ago

Ziqi-Li commented 11 months ago

Update to n_jobs = -1 as default to use all processors. And to fix typo and make clarification in the example notebook.

martinfleis commented 11 months ago

Thanks!

What about that pool keyword causing hard-breaking change (https://github.com/pysal/mgwr/pull/137#discussion_r1414313848)? That was the only point I though actually needs to be resolved, but these are nice to have as well!

Ziqi-Li commented 11 months ago

Thanks!

What about that pool keyword causing hard-breaking change (#137 (comment))? That was the only point I though actually needs to be resolved, but these are nice to have as well!

How about this one with the new commit?

image