Skip to content
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

Show example feature importance plot #310

Merged
merged 16 commits into from Nov 4, 2022
Merged

Show example feature importance plot #310

merged 16 commits into from Nov 4, 2022

Conversation

kelly-sovacool
Copy link
Member

@kelly-sovacool kelly-sovacool commented Nov 2, 2022

Issues

Change(s) made

  • Creates an example feature importance plot in the parallel vignette.
  • Slightly simplifies get_feature_importance() -- it doesn't need the train_data.

Checklist

(Strikethrough any points that are not applicable.)

  • [ ] Write unit tests for any new functionality or bug fixes.
  • Update docs if there are any API changes:
    • roxygen comments
    • vignettes
  • Update NEWS.md if this includes any user-facing changes.
  • The check workflow succeeds on your most recent commit. This is always required before the PR can be merged.

@kelly-sovacool kelly-sovacool requested review from a team and sklucas and removed request for a team November 3, 2022 18:41
@kelly-sovacool kelly-sovacool marked this pull request as ready for review November 4, 2022 17:51
@kelly-sovacool kelly-sovacool merged commit 5b2cc3a into main Nov 4, 2022
@kelly-sovacool kelly-sovacool deleted the feat-imp branch November 4, 2022 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add feature importance plot
2 participants