Skip to content

DOC add link to sklearn_is_fitted example in check_is_fitted #30621 #30995

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yetunthu90
Copy link

Reference Issues/PRs 30621

What does this implement/fix? Explain your changes.

DOC add link to sklearn_is_fitted example plot_kde_1d.py

Any other comments?

Copy link

❌ Linting issues

This PR is introducing linting issues. Here's a summary of the issues. Note that you can avoid having linting issues by enabling pre-commit hooks. Instructions to enable them can be found here.

You can see the details of the linting issues under the lint job here


black

black detected issues. Please run black . locally and push the changes. Here you can see the detected issues. Note that running black might also fix some of the issues which might be detected by ruff. Note that the installed black version is black=24.3.0.


--- /home/runner/work/scikit-learn/scikit-learn/sklearn/utils/validation.py	2025-03-14 13:46:02.412044+00:00
+++ /home/runner/work/scikit-learn/scikit-learn/sklearn/utils/validation.py	2025-03-14 13:46:27.333017+00:00
@@ -2965,6 +2965,5 @@
 
     if not no_val_X and check_params.get("ensure_2d", True):
         _check_n_features(_estimator, X, reset=reset)
 
     return out
-
would reformat /home/runner/work/scikit-learn/scikit-learn/sklearn/utils/validation.py

Oh no! 💥 💔 💥
1 file would be reformatted, 920 files would be left unchanged.

Generated for commit: 83144ca. Link to the linter CI: here

@StefanieSenger
Copy link
Contributor

Hello @yetunthu90, welcome to the repo. Are you planning to further work on that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants