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

export_strds: specify path to the directory where output is written #316

Merged
merged 1 commit into from
Mar 11, 2022

Conversation

metzm
Copy link
Contributor

@metzm metzm commented Mar 9, 2022

By default, t.rast.export in _export_strds() uses the current directory for raster exports which can be dangerous. In actinia, output should be written to a temporary directory in the current temporary GRASS db, as in e.g. _export_raster().

Important rule: if you don't know what the current working directory is, don't use it (esp. for large temporary files).

@metzm metzm requested a review from mmacata March 9, 2022 15:19
@metzm metzm added the bug Something isn't working label Mar 10, 2022
@metzm metzm merged commit 4604c4c into main Mar 11, 2022
@metzm metzm deleted the export_strds branch March 11, 2022 08:48
@mmacata mmacata added this to the 3.1.3 milestone Nov 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants