This is the first (big) step towards #40: It unifies the output of getTree() and treeInfo() and thus makes most of the {ranger} functionality redundant.
This PR is relatively large: the code base shrinks by 200 lines of code.
Note: The rds files for the vignette differed from the output, even for the CRAN version of the package. Thus, I have updated them first with the CRAN package of {randomForestExplainer}, and then compared with the output of this PR. The PR does not seem to change any values, and I have also updated these temp files with the PR version.
pbiecek
commented
8 months ago
This is the first (big) step towards #40: It unifies the output of
getTree()andtreeInfo()and thus makes most of the {ranger} functionality redundant.This PR is relatively large: the code base shrinks by 200 lines of code.
Note: The rds files for the vignette differed from the output, even for the CRAN version of the package. Thus, I have updated them first with the CRAN package of {randomForestExplainer}, and then compared with the output of this PR. The PR does not seem to change any values, and I have also updated these temp files with the PR version.