Skip to content

Commit

Permalink
Change n_estimators to sensible default value
Browse files Browse the repository at this point in the history
  • Loading branch information
ichkoar committed Jan 3, 2019
1 parent d9f7260 commit f41787d
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions dagging/_dagging.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
class BaseDagging(BaseEnsemble):
def __init__(self,
base_estimator=None,
n_estimators=3,
n_estimators=10,
voting='soft',
random_state=None):
super(BaseDagging, self).__init__(
Expand Down Expand Up @@ -114,7 +114,7 @@ class DaggingClassifier(BaseDagging, ClassifierMixin):

def __init__(self,
base_estimator=None,
n_estimators=3,
n_estimators=10,
voting='soft',
random_state=None):
super(DaggingClassifier, self).__init__(
Expand Down Expand Up @@ -225,7 +225,7 @@ class DaggingRegressor(BaseDagging, RegressorMixin):

def __init__(self,
base_estimator=None,
n_estimators=3,
n_estimators=10,
random_state=None):
super(DaggingRegressor, self).__init__(
base_estimator=base_estimator,
Expand Down
2 changes: 1 addition & 1 deletion dagging/tests/test_dagging.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,5 +9,5 @@ def test_check_dagging_classifier():


def test_check_dagging_regressor():
model = DaggingRegressor(random_state=0)
model = DaggingRegressor(random_state=0, n_estimators=2)
check_estimator(model)

0 comments on commit f41787d

Please sign in to comment.