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.

Sign up for GitHub

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

Jump to bottom

[FEA] Remove UserWarning from RandomForestClassifier #5224

Open
miguelusque opened this issue Feb 10, 2023 · 0 comments · May be fixed by #5233
Open

[FEA] Remove UserWarning from RandomForestClassifier #5224

miguelusque opened this issue Feb 10, 2023 · 0 comments · May be fixed by #5233
Labels
? - Needs Triage Need team to review and classify feature request New feature or request
Projects
Feature Planning

Comments

@miguelusque
Copy link
Member

Is your feature request related to a problem? Please describe.
When invoking the .fit() method in the RandomForestClassifier, the following UserWarning is displayed.

IMO, this UserWarning might be moved to the API documentation (https://docs.rapids.ai/api/cuml/stable/api.html#random-forest) and not displayed to the user in the python code execution.

/opt/conda/envs/rapids/lib/python3.9/site-packages/cuml/internals/api_decorators.py:794: UserWarning: For reproducible results in Random Forest Classifier or for almost reproducible results in Random Forest Regressor, n_streams=1 is recommended. If n_streams is > 1, results may vary due to stream/thread timing differences, even when random_state is set
return func(**kwargs)

Describe the solution you'd like
(see above)

Describe alternatives you've considered
You can always hide UserWarnings, but make your code a bit dirtier without need.
@miguelusque miguelusque added ? - Needs Triage Need team to review and classify feature request New feature or request labels Feb 10, 2023
@github-actions github-actions bot added this to Needs prioritizing in Feature Planning Feb 10, 2023
miguelusque added a commit to miguelusque/cuml that referenced this issue Feb 10, 2023
@miguelusque
Remove user warning about reproducible results
7b56684 
Fixes rapidsai#5224.
The content of this UserWarning will be moved to the documentation.
ajschmidt8 pushed a commit that referenced this issue Feb 13, 2023
@miguelusque
Remove user warning about reproducible results
ebe98d7 
Fixes #5224.
The content of this UserWarning will be moved to the documentation.
@miguelusque miguelusque linked a pull request Feb 14, 2023 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
? - Needs Triage Need team to review and classify feature request New feature or request
Projects
Feature Planning
Needs prioritizing
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove unneeded warning from Random Forest Classifier / Regressor miguelusque/cuml
1 participant
@miguelusque