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

Correct some of the Scikit-learn stubs #311

Merged
merged 3 commits into from
Jun 6, 2024

Conversation

ghislainpiot
Copy link
Contributor

Hello,

While we were in the process of using this repository, we found a few mistakes in the scikit-learn stubs.

These modifications are compatible with the version of scikit-learn used for the stubs.

Have a nice day,

@debonte
Copy link
Contributor

debonte commented May 7, 2024

@ghislainpiot, thanks for these fixes! I'll merge this as soon as you accept the CLA.

@ghislainpiot
Copy link
Contributor Author

@microsoft-github-policy-service agree company="SonarSource"

@ghislainpiot
Copy link
Contributor Author

ghislainpiot commented Jun 6, 2024

@debonte Sorry for the delay, I have accepted the CLA.

@debonte debonte merged commit d3462bd into microsoft:main Jun 6, 2024
3 checks passed
@debonte
Copy link
Contributor

debonte commented Jun 6, 2024

Merged. Thanks for the fix @ghislainpiot!

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