Add a new argument include_model to the as.data.frame() S3 methods for pioneer_dea and pioneer_mlm S3 objects. If set to FALSE, the columns from x$model should not be included. This will make it easier to understand the code in app, as we exclude these columns from the table for Malmquist. Instead, we should have a call to as.data.frame() with include_model set to FALSE. The default value should be TRUE as the current behaviour would be the expected from the R users perspective.
Add a new argument
include_model
to theas.data.frame()
S3 methods forpioneer_dea
andpioneer_mlm
S3 objects. If set toFALSE
, the columns fromx$model
should not be included. This will make it easier to understand the code in app, as we exclude these columns from the table for Malmquist. Instead, we should have a call toas.data.frame()
withinclude_model
set toFALSE
. The default value should beTRUE
as the current behaviour would be the expected from the R users perspective.