Unverified Commit 69d02e38 authored by david-cortes's avatar david-cortes Committed by GitHub
Browse files

[R-package] Make returned feature importances from lgb.importance() visible by default (#4194)



* Return feature importances as visible

Feature importances are returned as invisible. I don't get why would anyone want them invisible. This PR fixes it.

* Update R-package/R/lgb.importance.R

Thanks, didn't realize that, changed now.
Co-authored-by: default avatarJames Lamb <jaylamb20@gmail.com>
Co-authored-by: default avatarJames Lamb <jaylamb20@gmail.com>
parent 7ea2bc4d
......@@ -75,6 +75,7 @@ lgb.importance <- function(model, percentage = TRUE) {
)]
}
return(tree_imp_dt)
# adding an empty [] to ensure the table is printed the first time print.data.table() is called
return(tree_imp_dt[])
}
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment