changes prior to resubmit

NEWS.md

@@ -1,7 +1,7 @@
 # rMIDAS 0.4
 
 * `python` argument in `set_python_env` renamed to `x` for clarity
-* Minor fixes including bug in `complete()` function
+* Minor fixes including remedying bug in `complete()` function
 * Improved documentation
 
 # rMIDAS 0.3

R/load_utils.R

@@ -18,13 +18,19 @@ skip_if_no_numpy <- function() {
 #' @return `NULL`
 python_configured <- function() {
 
-  py_dep <- c("matplotlib","numpy","pandas","tensorflow","sklearn","os","random", "tensorflow_addons")
-  dep_avail <- sapply(py_dep, function (x) reticulate::py_module_available(x))
-
-  if (sum(dep_avail) == length(py_dep)) {
-    return(TRUE)
-  } else {
+  if (!reticulate::py_available()) {
     return(FALSE)
+  } else {
+
+    py_dep <- c("matplotlib","numpy","pandas","tensorflow","sklearn","os","random", "tensorflow_addons")
+    dep_avail <- sapply(py_dep, function (x) reticulate::py_module_available(x))
+
+    if (sum(dep_avail) == length(py_dep)) {
+      return(TRUE)
+    } else {
+      return(FALSE)
+    }
+
   }
 }
 

cran-comments.md

@@ -1,8 +1,10 @@
-## Resubmission
+## Resubmission 
 
 This is a resubmission. In this version I have:
 
-* Fixed the invalid URL to point to the moved content
+* Updated system checking functions to ensure all requirements are met before running examples that interface with Python
+* Added SystemRequirements to Description file
+* Fixed invalid URL to point to the moved content
 
 ## v0.4.0
 

inst/examples/overimputation.R

@@ -1,5 +1,5 @@
 \dontrun{
-# Run where Python available and configured correctly
+# Run where Python initialised and configured correctly
 if (python_configured()) {
 
 raw_data <- data.table(a = sample(c("red","yellow","blue",NA),1000, replace = TRUE),