correct interpretation of add_tailor(prop) (#248)

also corrects documentation on default value.

R/fit.R

@@ -103,10 +103,12 @@ make_inner_split <- function(object, data) {
       class = if (is.null(method)) "mc_split" else method
     )
 
+  # add_tailor(prop) is the proportion to train the postprocessor, while
+  # rsample::mc_cv(prop) is the proportion to train the model (#247)
   prop <- object$post$actions$tailor$prop
   rsample::inner_split(
     mocked_split,
-    list(prop = if (is.null(prop)) 2/3 else prop)
+    list(prop = if (is.null(prop)) 2/3 else 1 - prop)
   )
 }
 

R/post-action-tailor.R

@@ -20,7 +20,7 @@
 #' @param prop The proportion of the data in [fit.workflow()] that should be
 #' held back specifically for estimating the postprocessor. Only relevant for
 #' postprocessors that require estimation---see section Data Usage below to
-#' learn more. Defaults to 2/3.
+#' learn more. Defaults to 1/3.
 #'
 #' @param method The method with which to split the data in [fit.workflow()],
 #' as a character vector. Only relevant for postprocessors that