tidymodels / workflows

Modeling Workflows
https://workflows.tidymodels.org/
Other
205 stars 23 forks source link

revisit why `print.workflow()` re-implements print methods for its stages #236

Open simonpcouch opened 4 months ago

simonpcouch commented 4 months ago

workflows rewrites print methods for each of its stage for printing. We should revisit why this is the case, and either 1) document in a code comment why it is that we're rewriting those methods, or 2) re-implement the postprocessor printing in print.workflow()

https://github.com/tidymodels/workflows/blob/9e6ca982f8cd381cf4601cceb64f5ad323737476/R/workflow.R#L473-L479

Notably, the oddity with tailor printing's output currently is that, when in a trained workflow, both the workflow and its postprocessor stage have a tag marking themselves as "[trained]".