Skip to content

Commit b87bd0f

Browse files
committed
Add context manager
Context manager is added to code to prevent design() from exiting due to division warning that should be ignored.
1 parent 22d4fc4 commit b87bd0f

2 files changed

Lines changed: 9 additions & 9 deletions

File tree

R/update.R

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -108,9 +108,9 @@ update.dgp <- function(object, X, Y, refit = FALSE, reset = FALSE, verb = TRUE,
108108
}
109109
N0 <- constructor_obj_cp$N
110110
if ( identical(cores,as.integer(1)) ){
111-
constructor_obj_cp$train(N, ess_burn, disable)
111+
with(pkg.env$np$errstate(divide = 'ignore'), constructor_obj_cp$train(N, ess_burn, disable))
112112
} else {
113-
constructor_obj_cp$ptrain(N, ess_burn, disable, cores)
113+
with(pkg.env$np$errstate(divide = 'ignore'), constructor_obj_cp$ptrain(N, ess_burn, disable, cores))
114114
}
115115
burnin <- as.integer(N0 + 0.75*N)
116116
} else {
@@ -185,7 +185,7 @@ update.gp <- function(object, X, Y, refit = FALSE, reset = FALSE, verb = TRUE, .
185185

186186
if ( refit ){
187187
if ( verb ) message("Re-fitting ...", appendLF = FALSE)
188-
constructor_obj_cp$train()
188+
with(pkg.env$np$errstate(divide = 'ignore'), constructor_obj_cp$train())
189189
if ( verb ) message(" done")
190190
}
191191

R/utils.R

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -945,7 +945,7 @@ crop <- function(object, crop_id_list, refit_cores, verb) {
945945
struc$para_path <- matrix(c(struc$scale, struc$length, struc$nugget), nrow = 1, byrow=T)
946946
}
947947
obj <- pkg.env$dgpsi$gp(X, Y, struc)
948-
obj$train()
948+
with(pkg.env$np$errstate(divide = 'ignore'), obj$train())
949949
res <- list()
950950
res[['id']] <- object$id
951951
res[['data']][['X']] <- X
@@ -979,7 +979,7 @@ crop <- function(object, crop_id_list, refit_cores, verb) {
979979
struc$para_path <- matrix(c(struc$scale, struc$length, struc$nugget), nrow = 1, byrow = T)
980980
}
981981
obj <- pkg.env$dgpsi$gp(X, Y[,j,drop=F], struc)
982-
obj$train()
982+
with(pkg.env$np$errstate(divide = 'ignore'), obj$train())
983983
res_j <- list()
984984
res_j[['id']] <- uuid::UUIDgenerate()
985985
res_j[['data']][['X']] <- X
@@ -1020,9 +1020,9 @@ crop <- function(object, crop_id_list, refit_cores, verb) {
10201020
object$constructor_obj$update_all_layer(all_layer)
10211021
if ( verb ) message(" - Re-fitting ...", appendLF = FALSE)
10221022
if ( identical(refit_cores, as.integer(1)) ){
1023-
object$constructor_obj$train(as.integer(100), as.integer(10), TRUE)
1023+
with(pkg.env$np$errstate(divide = 'ignore'), object$constructor_obj$train(as.integer(100), as.integer(10), TRUE))
10241024
} else {
1025-
object$constructor_obj$ptrain(as.integer(100), as.integer(10), TRUE, refit_cores)
1025+
with(pkg.env$np$errstate(divide = 'ignore'), object$constructor_obj$ptrain(as.integer(100), as.integer(10), TRUE, refit_cores))
10261026
}
10271027
est_obj <- object$constructor_obj$estimate()
10281028
internal_dims <- object[['specs']][['internal_dims']]
@@ -1058,9 +1058,9 @@ crop <- function(object, crop_id_list, refit_cores, verb) {
10581058
object$constructor_obj$update_all_layer(all_layer)
10591059
if ( verb ) message(" - Re-fitting ...", appendLF = FALSE)
10601060
if ( identical(refit_cores, as.integer(1)) ){
1061-
object$constructor_obj$train(as.integer(100), as.integer(10), TRUE)
1061+
with(pkg.env$np$errstate(divide = 'ignore'), object$constructor_obj$train(as.integer(100), as.integer(10), TRUE))
10621062
} else {
1063-
object$constructor_obj$ptrain(as.integer(100), as.integer(10), TRUE, refit_cores)
1063+
with(pkg.env$np$errstate(divide = 'ignore'), object$constructor_obj$ptrain(as.integer(100), as.integer(10), TRUE, refit_cores))
10641064
}
10651065
est_obj <- object$constructor_obj$estimate()
10661066
internal_dims <- object[['specs']][['internal_dims']]

0 commit comments

Comments
 (0)