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

添加CascadeForestClassifier类对cross_val_score交叉验证的支持。 #8

Merged
merged 5 commits into from
Feb 3, 2021

Conversation

pjgao
Copy link
Contributor

@pjgao pjgao commented Feb 1, 2021

BaseCascadeForest添加基类BaseEstimator以包含get_params方法; 对CascadeForestClassifier添加ClassifierMixin基类以包含score方法。

Related Issue: #2

给`BaseCascadeForest`添加基类`BaseEstimator`以包含`get_params`方法; 对`CascadeForestClassifier`添加`ClassifierMixin`基类以包含`score`方法。
@xuyxu
Copy link
Member

xuyxu commented Feb 2, 2021

Hi @pjgao, thanks for the PR! It looks like there is a problem on flake8 formatting, you can check the logs from the CI: Code-Quality for details.

In addition, do not forget to add your contribution and name in CHANGELOG.rst ;-)

@xuyxu
Copy link
Member

xuyxu commented Feb 2, 2021

Great! I will have a final check tomorrow, and merge it if there is no problem ;-)

Once again, thanks for the PR!

@xuyxu
Copy link
Member

xuyxu commented Feb 3, 2021

Hi @pjgao, I have made some edits on your code:

  • Sync with the master branch
  • Re-write the __init__ for CascadeForestClassifier to make get_params() and set_params() actually work

Let me know if you are OK with these changes ;-) I will merge this PR if you have no problem.

@pjgao
Copy link
Contributor Author

pjgao commented Feb 3, 2021

It looks great, now we can use <tab> to autocomplete the params.

@xuyxu xuyxu merged commit 08debc0 into LAMDA-NJU:master Feb 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants