Skip to content

Commit

Permalink
FIX: hyperband now supported with stacking
Browse files Browse the repository at this point in the history
  • Loading branch information
patr1ckm committed Aug 2, 2018
1 parent a9cc657 commit e5320d9
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,11 @@
# Changelog

## [1.5.1] - Unreleased

### Changed

- Hyperband is now supported for stacking estimators in `civis_ml`. Fixes #131.

## [1.5.0] - 2018-05-20

### Changed
Expand Down
3 changes: 1 addition & 2 deletions R/civis_ml.R
Original file line number Diff line number Diff line change
Expand Up @@ -590,8 +590,7 @@ create_and_run_model <- function(file_id = NULL,

if (!is.null(cross_validation_parameters)) {
hyperband <- identical(cross_validation_parameters, "hyperband")
hyperband_not_supported <- model_type %in% c("sparse_logistic", "sparse_linear_regressor",
"stacking_regressor")
hyperband_not_supported <- model_type %in% c("sparse_logistic", "sparse_linear_regressor")
if (hyperband & hyperband_not_supported) {
stop(paste0("cross_validation_parameters = \"hyperband\" not supported for ", model_type))
}
Expand Down

0 comments on commit e5320d9

Please sign in to comment.