You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Provide clear naming for the inconsistent ways to get the cache directory #1124
importopenmlopenml.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:
we make them both functions so they are accessed in the same way (and using a function will make a get\set pattern easier where a set can be optionally for the session or saved to the configuration file)
make it clear in the name what (part of) the cache directory is returned
Activity
mfeurer commentedon Feb 21, 2022
I agree on both, we should provide getters and setters for stuff from the config, and we should make the naming clearer.
Fix #1124
Fix 1124: provide clear naming for cache directories (#1254)