Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use standard Linux locations for config and cache #884

Closed
mitar opened this issue Nov 13, 2019 · 2 comments · Fixed by #987
Closed

Use standard Linux locations for config and cache #884

mitar opened this issue Nov 13, 2019 · 2 comments · Fixed by #987
Assignees

Comments

@mitar
Copy link
Member

mitar commented Nov 13, 2019

Currently OpenML package creates .openml in home directory. These days this is not a nice behavior, because there is a standard where things should go.

  • Use XDG_CONFIG_HOME environment variable to determine base config directory.
  • Use XDG_CACHE_HOME environment variable to determine base cache directory.

See more practical information and details here.

So, by default, config should go into ~/.config/openml and cache into ~/.cache/openml.

@mfeurer
Copy link
Collaborator

mfeurer commented Nov 18, 2019

Thank you for the pointers, this makes total sense.

So far, we have agreed to keep the config and cache consistent across languages, so I'd like to hear @giuseppec and @janvanrijn's opinion for R and Java.

@HeidiSeibold
Copy link
Member

@mllg @berndbischl @giuseppec this is likely important for the R package as well. 👆

@mwever mwever self-assigned this Oct 30, 2020
mfeurer added a commit that referenced this issue Jan 28, 2021
PGijsbers pushed a commit that referenced this issue Feb 10, 2021
* Fix #883 #884 #906 #972

* Address Mitar's comments

* rework for Windows/OSX, some mypy pleasing due to pre-commit

* type fixes and removing unused code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants