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

Docs/sim notebook refactor #65

Merged
merged 6 commits into from
Sep 16, 2020
Merged

Docs/sim notebook refactor #65

merged 6 commits into from
Sep 16, 2020

Conversation

jason-bentley
Copy link
Contributor

This PR

This PR adds a refactored simulated data-based FACET classification tutorial that explores the following:

  1. The impact of correlation/interaction between features and how synergy and redundancy changes
  2. How synergy/redundancy change with max_depth in a random forest

Note that because the simulations take some time, the data used for the results presented have been added to the tutorials folder, so when documentation is rebuilt, we do not need to wait for simulations and ensure we always can regenerate the exact same figures.

To run
Navigate to the sphinx folder and run python make.py html.

Expected outputs
Examples of the expected output are below.
image

image

image

@jason-bentley jason-bentley added the documentation Improvements or additions to documentation label Sep 16, 2020
@jason-bentley jason-bentley self-assigned this Sep 16, 2020
Copy link
Member

@j-ittner j-ittner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@j-ittner j-ittner merged commit cac962a into develop Sep 16, 2020
@j-ittner j-ittner deleted the docs/sim_notebook_refactor branch September 23, 2020 16:43
@j-ittner j-ittner added this to the 1.0.1 milestone Mar 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants