Evovest/EvoTrees.jl

Update MLJ feature importance access to current standard

john-waczak opened this issue · 1 comments

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.

Fixed with #167