Update MLJ feature importance access to current standard
john-waczak opened this issue · 1 comments
john-waczak commented
Currently feature importances from EvoTrees are accessed via report()
. We should update the interface to comply with the new method defined here: https://alan-turing-institute.github.io/MLJ.jl/dev/adding_models_for_general_use/#Feature-importances. In particular I suggest we do the following:
- remove automatically computed feature importances from
report()
- dispatch
MLJModelInterface.feature_importances(model::M, fitresult, report)
to compute feature importances as desired - dispatch
MLJModelInterface.reports_feature_importances(::Type{<:M}) = true
for each EvoTrees model so MLJ knows we can access feature importances.