mlr-org/mlrMBO

setMBOControlTermination lists wrong(?) parameter type

mb706 opened this issue · 4 comments

mb706 commented

https://mlrmbo.mlr-org.com/reference/setMBOControlTermination.html lists time.budget and time.execution.budget type both as integer(1), but there is no reason why this should not be numeric, right?

Actually we store it as numeric because it supports Inf. So yes

just change it to numeric(1)

mb706 commented

Actually we store it as numeric because it supports Inf.

but otherwise fractional seconds are not supported?

They are. difftime returns fractions of seconds as well.