search

openml / openml-python

Python module to interface with OpenML
https://openml.github.io/openml-python/main/
Other
279 stars 143 forks source link

[Refactor] `openml/config`, switch from globals to just using a classes? #1300

Closed eddiebergman closed 9 months ago

eddiebergman commented 9 months ago

There's a lot of jumping around with global and it's hard to keep track of, why use globals here?

PGijsbers commented 9 months ago

I think because it was the least invasive change at the time. If we do change this, we need to give users warnings and time to accommodate (as with all breaking changes).

eddiebergman commented 9 months ago

I think that's too breaking and would add a lot of cruft to deal with. Will close as it's not worth it