Merge 8a5b87a into ab7a359

Author: rok-cesnovar

NEWS.md

@@ -5,6 +5,9 @@
 * Fixed bug that caused stdour/stderr not being read at the end of
 optimization. (#522)
 
+* Fixed issue with handling `NA` as the reported external process
+status. (#544, @myshkin)
+
 ### New features
 
 * Default directory changed to `.cmdstan` instead of `.cmdstanr` so that

R/model.R

@@ -513,7 +513,7 @@ compile <- function(quiet = TRUE,
     },
     error_on_status = FALSE
   )
-  if (run_log$status != 0) {
+  if (is.na(run_log$status) || run_log$status != 0) {
     stop("An error occured during compilation! See the message above for more information.",
          call. = FALSE)
   }
@@ -676,7 +676,7 @@ check_syntax <- function(pedantic = FALSE,
     },
     error_on_status = FALSE
   )
-  if (run_log$status != 0) {
+  if (is.na(run_log$status) || run_log$status != 0) {
     stop("Syntax error found! See the message above for more information.",
          call. = FALSE)
   }

R/run.R

@@ -499,7 +499,7 @@ CmdStanRun$set("private", name = "run_variational_", value = .run_other)
     stdout = stdout_file,
     error_on_status = FALSE
   )
-  if (ret$status != 0) {
+  if (is.na(ret$status) || ret$status != 0) {
     if (file.exists(stdout_file)) {
       cat(readLines(stdout_file), sep = "\n")
     }