Skip to content
This repository has been archived by the owner on Dec 6, 2023. It is now read-only.

Support for max_features #27

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Support for max_features #27

wants to merge 6 commits into from

Conversation

a-n-ermakov
Copy link
Contributor

#26 implementation

Backward compatible with previous version

@a-n-ermakov
Support of parameters max_features (for threshold exceeding) and bo…
cf9bd6d 
…otstrap_max_features (for `bootstrap_threshold` exceeding) while performing threshold check on bootstrap step (SelectFromModel invoke) and while performing final check (get_support method). Backward compatible with previous implementation
@coveralls
Copy link

coveralls commented Mar 15, 2019 •
edited
Loading

Pull Request Test Coverage Report for Build 85

  • 55 of 55 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.3%) to 97.642%
Totals Coverage Status
Change from base Build 79: 0.3%
Covered Lines: 414
Relevant Lines: 424
💛 - Coveralls

@a-n-ermakov
Copy link
Contributor Author

According to this stackoverflow question, travis CI has a limit for log of 4MB

https://stackoverflow.com/questions/30463993/is-there-a-limitation-on-the-log-lines-in-a-travis-build

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@a-n-ermakov @coveralls