openml / openml-python

OpenML's Python API for a World of Data and More 💫
http://openml.github.io/openml-python/
Other
279 stars 143 forks source link

Provide clear naming for the inconsistent ways to get the cache directory #1124

Closed PGijsbers closed 1 year ago

PGijsbers commented 2 years ago
import openml
openml.config.get_cache_directory()   # is:   '~\\.openml\\org\\openml\\www'
openml.config.cache_directory         # is:   '~\\.openml'

get_cache_directory gets the cache for the currently configured server, while cache_directory gets the root of the cache directory. set_cache_directory(str) configures the root.

If we plan to maintain an external interface for both types, I propose we:

mfeurer commented 2 years ago

I agree on both, we should provide getters and setters for stuff from the config, and we should make the naming clearer.