Skip to content

Random forest eigen values - #37

Open
wagenrace wants to merge 12 commits into
cytodata:masterfrom
wagenrace:random_forest_eigen_values
Open

Random forest eigen values#37
wagenrace wants to merge 12 commits into
cytodata:masterfrom
wagenrace:random_forest_eigen_values

Conversation

@wagenrace

@wagenrace wagenrace commented Jan 13, 2020

Copy link
Copy Markdown
Collaborator

The following pull request adds, fixes, and/or modifies:

Explored eigen values with random forests.
@gwaygenomics and @koalive, I really can use your help here. The results get worst by adding eigen values to the mix even if I only pick the ones with a significant different in the targets.
But I cannot really explain why.
So far I have 2 hypothesizes:

  • There is a bug some where
  • The extra features are still to noisy making it so the trees overfit

I performed the following prior to filing this pull request:

  • Tested that my change does not break the analysis pipeline
  • Ran a linter through my code
  • Update environment dependencies if my code introduces a new package

@wagenrace
wagenrace requested a review from gwaybio January 13, 2020 20:09
@wagenrace wagenrace added the help wanted Extra attention is needed label Jan 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant