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

How to specify the folder path for the SQLite file? #549

Closed
tomfelder94 opened this issue Jul 17, 2024 · 4 comments · Fixed by #550
Closed

How to specify the folder path for the SQLite file? #549

tomfelder94 opened this issue Jul 17, 2024 · 4 comments · Fixed by #550
Assignees

Comments

@tomfelder94
Copy link

Description of the issue

I would like to saved the generated SQLite file at a specific location (a shared SSD drive), not under $HOME/.open-MaStR/data/sqlite. Is there a way to provide an absolute path as an argument when initializing the Mastr object? Or do I have to manually copy it after each download?

@nesnoj
Copy link
Collaborator

nesnoj commented Jul 17, 2024

Hey @tomfelder94!
Yes, this feature was added in v0.14.0 (PR #402). You can change it by adjusting the env variable OUTPUT_PATH like

os.environ['OUTPUT_PATH'] = "/your/custom/output_path"

which is read by this function.

According to #402 this hint was part of the docs but somehow vanished.. 🤔 - do you know why @FlorianK13 ?

cf. #520

@FlorianK13
Copy link
Member

Hi @nesnoj I don't know why this is not in the docs. We definitely should add it somewhere because this question comes up regularly.

@nesnoj
Copy link
Collaborator

nesnoj commented Jul 18, 2024

Hi @nesnoj I don't know why this is not in the docs. We definitely should add it somewhere because this question comes up regularly.

Ok, I'll take care..

@nesnoj
Copy link
Collaborator

nesnoj commented Jul 23, 2024

Fixed in #550, docs will be updated in next release @tomfelder94

@nesnoj nesnoj closed this as completed Jul 23, 2024
This was referenced Oct 11, 2024
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.

3 participants