Commit 04f3f4d
feat(EstimatorReport): Display the feature permutation importance and mean decrease impurity (#1365)
Closes #1319 and #1323
Todo:
- ~[ ] Add example~ deferred to #1390
- [x] Coverage at 100%
- [x] Add API docs
- [x] Check for `# TODO` in code
- [x] Check what happens if `scoring` is a callable (is it cached?)
- scoring was not included in the cache key at all! Fixed
- [x] Check what happens if `random_state` is a `RandomState` instance
(is it cached?)
- If `random_state` is a RandomState then the call should not be cached,
because reusing a `RandomState` gives a different result.
---------
Co-authored-by: Sylvain Combettes <[email protected]>1 parent bc1f56d commit 04f3f4d
File tree
16 files changed
+1515
-79
lines changed- examples
- getting_started
- use_cases
- skore
- src/skore
- sklearn/_estimator
- utils
- tests/unit
- item
- sklearn/estimator/feature_importance
- utils
- sphinx
- reference/report
16 files changed
+1515
-79
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
32 | 32 | | |
33 | 33 | | |
34 | 34 | | |
35 | | - | |
| 35 | + | |
36 | 36 | | |
37 | 37 | | |
38 | 38 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
15 | | - | |
| 15 | + | |
| 16 | + | |
16 | 17 | | |
17 | 18 | | |
18 | 19 | | |
| |||
82 | 83 | | |
83 | 84 | | |
84 | 85 | | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
85 | 93 | | |
86 | 94 | | |
87 | 95 | | |
88 | 96 | | |
89 | | - | |
| 97 | + | |
| 98 | + | |
90 | 99 | | |
91 | 100 | | |
92 | 101 | | |
| |||
0 commit comments