New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Report confidence interval for permutation feature importance #326
Conversation
Otherwise, check() won't be able to find it. https://www.r-bloggers.com/2021/06/documentation-for-internal-functions/
Also mark find_permuted_perf_metric internal so doc will be created for developers to read.
Codecov ReportBase: 98.18% // Head: 98.19% // Increases project coverage by
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more Additional details and impacted files@@ Coverage Diff @@
## main #326 +/- ##
=======================================
Coverage 98.18% 98.19%
=======================================
Files 13 13
Lines 1103 1109 +6
=======================================
+ Hits 1083 1089 +6
Misses 20 20
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
To fix this warning: Use of .data in tidyselect expressions was deprecated in tidyselect 1.2.0. i Please use `all_of(var)` (or `any_of(var)`) instead of `.data[[var]]`
Because `names()` is a base R function already, and it's not obvious that `names` referred to the feature or group of correlated features.
- `names` renamed to `feat` - `lower` and `upper` for bounds of 95% CI
Issues
Change(s) made
names
tofeat
for clarity and to avoid clashing with the basenames()
function.Checklist
(
Strikethroughany points that are not applicable.)NEWS.md
if this includes any user-facing changes.