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

Add compress option to saveProject #73

Merged
merged 3 commits into from
Nov 19, 2020
Merged

Add compress option to saveProject #73

merged 3 commits into from
Nov 19, 2020

Conversation

pralitp
Copy link
Contributor

@pralitp pralitp commented Nov 5, 2020

Given saveProject runs incredibly slow on PIC's lustre filesystem and that they do not recommend storing files compressed there we make
the compress option, which gets passed on to base::save configurable
in saveProject.
Also given we call this method indirectly from so many methods if not explicitly provided
then we check the global options options("rgcam.saved_compressed") and if
that is not set we fall back to the default behavior which continues to be "xz".

that they *do not* reccommend storing files compressed there we make
the `compress` option, which gets passed on to `base::save` configurable
in `saveProject`.
Also given we call this method indirectly from so many methods if not explicitly provided
then we check the global options `options("rgcam.saved_compressed")` and if
that is not set we fall back to the default behavior which continues to be "xz".
@codecov
Copy link

codecov bot commented Nov 15, 2020

Codecov Report

Merging #73 (5fe9ea0) into master (4f744f7) will increase coverage by 0.77%.
The diff coverage is 100.00%.

Impacted Files Coverage Δ
R/manageProjectData.R 90.07% <100.00%> (+1.09%) ⬆️
R/querymi.R 86.72% <100.00%> (+1.76%) ⬆️

@pralitp pralitp merged commit 7e678fa into master Nov 19, 2020
@pralitp pralitp deleted the save-compress-option branch November 19, 2020 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants