Open mb706 opened 2 years ago
potential problem: we don't want to tempt people into doing the wrong kind of preprocessing on train and test set
suggestion (bb):
outdata = preproc("encode", indata, paramlist, state = state)
maybe we want to have preproc(po("encode", paramlist))
instead, more consistent and would also work with PipeOps that are not in po()
.
could also overload the %>>%
operator here.
maybe: whenever state is given, or when optional var predict = TRUE
(default FALSE) do prediction, otherwise training. We live with the fact that the state gets saved to the 1st arg pipeop. No reason not to make this work with graphs as well.