Skip to content

Commit

Permalink
handle requirements as in current paradox
Browse files Browse the repository at this point in the history
  • Loading branch information
mb706 committed Jan 14, 2024
1 parent df23a38 commit bf78002
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions R/ParamSet.R
Original file line number Diff line number Diff line change
Expand Up @@ -294,12 +294,12 @@ ParamSet = R6Class("ParamSet",
}

if (check_strict) {
required = setdiff(self$ids(tags = "required"), ns)
if (length(required) > 0L) {
return(sprintf("Missing required parameters: %s", str_collapse(required)))
}
return(self$check_dependencies(xs))
## required = setdiff(self$ids(tags = "required"), ns)
## if (length(required) > 0L) {
## return(sprintf("Missing required parameters: %s", str_collapse(required)))
## }
if (!self$test_constraint(xs, assert_value = FALSE)) return(sprintf("Constraint not fulfilled."))
return(self$check_dependencies(xs))
}

TRUE # we passed all checks
Expand Down

0 comments on commit bf78002

Please sign in to comment.