Skip to content

Commit

Permalink
cont. merge with master
Browse files Browse the repository at this point in the history
  • Loading branch information
adokter committed Jun 7, 2021
1 parent 034003c commit 3aab665
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 11 deletions.
9 changes: 3 additions & 6 deletions R/calculate_vp.R
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@
#' are selected automatically, and other user settings are ignored.
#' @param verbose logical. When TRUE, pipe Docker stdout to R console. On
#' Windows always TRUE.
#' @param warnings logical. When TRUE, pipe vol2bird warnings to R console.
#' @param mount character. String with the mount point (a directory path) for
#' the Docker container.
#' @param sd_vvp_threshold numeric. Lower threshold in radial velocity standard
Expand Down Expand Up @@ -192,7 +191,7 @@
#' # clean up:
#' file.remove("~/volume.h5")
calculate_vp <- function(file, vpfile = "", pvolfile_out = "",
autoconf = FALSE, verbose = FALSE, warnings = TRUE,
autoconf = FALSE, verbose = FALSE,
mount = dirname(file[1]), sd_vvp_threshold,
rcs = 11, dual_pol = TRUE, rho_hv = 0.95, elev_min = 0,
elev_max = 90, azim_min = 0, azim_max = 360,
Expand Down Expand Up @@ -257,8 +256,6 @@ calculate_vp <- function(file, vpfile = "", pvolfile_out = "",

assert_that(is.flag(verbose))

assert_that(is.flag(warnings))

assert_that(is.writeable(mount))

if (!missing(sd_vvp_threshold)) {
Expand Down Expand Up @@ -440,13 +437,13 @@ calculate_vp <- function(file, vpfile = "", pvolfile_out = "",
# on mac and linux:
if (missing(local_install)) {
result <- system(docker_command,
ignore.stdout = !verbose, ignore.stderr = !warnings
ignore.stdout = !verbose
)
}
else {
# using a local install of vol2bird:
result <- system(paste("bash -l -c \"", local_install, file, profile.tmp, pvolfile_out, "\""),
ignore.stdout = !verbose, ignore.stderr = !warnings)
ignore.stdout = !verbose)
}
} else {
# on Windows platforms:
Expand Down
5 changes: 0 additions & 5 deletions man/calculate_vp.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 3aab665

Please sign in to comment.