Skip to content

Commit

Permalink
fix #6 change pi_name param to slurm_account in proof_start
Browse files Browse the repository at this point in the history
  • Loading branch information
sckott committed Jan 25, 2024
1 parent 436e006 commit 11914b3
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 15 deletions.
19 changes: 11 additions & 8 deletions R/start.R
Original file line number Diff line number Diff line change
@@ -1,21 +1,24 @@
#' Start PROOF Cromwell server
#'
#' @export
#' @param pi_name (character) PI name in the form last_f; only needed if user
#' is in more than one SLURM account
#' @references <https://github.com/FredHutch/proof-api#post-cromwell-server>
#' @param slurm_account (character) PI name in the form `last_f`, where last
#' is the last name of the PI and f is the first name, e.g., "Jane Doe"
#' would be `doe_j`; only needed if user is in more than one SLURM account.
#' optional
#' @references <https://github.com/FredHutch/proof-api/#post-cromwell-server>
#' @details Does not return PROOF/Cromwell server URL, for that you have to
#' periodically call [proof_status()], or wait for the email from the
#' PROOF API
#' PROOF API. See the link to proof-api for more details about the
#' slurm account.
#' @inheritSection proof_status Timeout
#' @return A list with fields:
#' - `job_id` (character)
#' - `info` (character)
proof_start <- function(pi_name = NULL) {
#' - `job_id` (character) - the job ID
#' - `info` (character) - message
proof_start <- function(slurm_account = NULL) {
response <- POST(
make_url("cromwell-server"),
proof_header(),
body = list(pi_name = pi_name),
body = list(slurm_account = slurm_account),
encode = "json",
timeout(proofr_env$timeout_sec)
)
Expand Down
17 changes: 10 additions & 7 deletions man/proof_start.Rd

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

0 comments on commit 11914b3

Please sign in to comment.