-
Notifications
You must be signed in to change notification settings - Fork 1
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
Documentation #71
Documentation #71
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1.a I guess the branch name could follow the contribuition guilines x/issue_name
, where x
represents the issue
@@ -21,7 +39,19 @@ def __init__(self, model_file=None, n_estimators=100, max_depth=None, random_sta | |||
self.random_state = random_state | |||
self.model = RandomForestClassifier(n_estimators=self.n_estimators, max_depth=self.max_depth, random_state=self.random_state) | |||
|
|||
def train_and_get_scores(self, df, concatened_column_name, all_features_columns, test_size=0.3): | |||
def train_and_get_scores(self, df, concatened_column_name, all_features_columns, test_size=0.3): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1.b need to undo the identation changes in order to run this code and pass the tests
|
|
|
Codecov ReportBase: 79.89% // Head: 79.89% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #71 +/- ##
=======================================
Coverage 79.89% 79.89%
=======================================
Files 31 31
Lines 965 965
=======================================
Hits 771 771
Misses 194 194
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. |
- Insert supported by - Remove list of tables and index - Insert logo in header - Insert link to github organization - update doxygen .xml
9e0b9a6
to
9925063
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Show!!
Motivation
Updates all project documentation, including readme and more general structures
Changes