Skip to content

Fix #30621 Added reference to plot_bias_variance.py in learning_curve.rst#30845

Closed
RayyanShaikh1 wants to merge 1 commit intoscikit-learn:mainfrom
RayyanShaikh1:plot_bias_variance_update
Closed

Fix #30621 Added reference to plot_bias_variance.py in learning_curve.rst#30845
RayyanShaikh1 wants to merge 1 commit intoscikit-learn:mainfrom
RayyanShaikh1:plot_bias_variance_update

Conversation

@RayyanShaikh1
Copy link
Copy Markdown

Fix #30621

What does this implement/fix? Explain your changes.

Added reference to plot_bias_variance.py in learning_curve.rst

@github-actions
Copy link
Copy Markdown

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 5496028. Link to the linter CI: here

Copy link
Copy Markdown
Member

@StefanieSenger StefanieSenger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @RayyanShaikh1,

thanks for your contribution, but I'm afraid I need to close this PR, because it is not what we're looking to do here. If you want to contribute to this issue, please work thoroughly through other PRs that had been successfully merged to find out more about it.

bug_triaging
maintainer
plotting
learning_curve No newline at end of file
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
learning_curve

No, we don't need to add this section to the index list of the developer guide.

* :ref:`sphx_glr_auto_examples_model_selection_plot_train_error_vs_test_error.py`
* :ref:`sphx_glr_auto_examples_model_selection_plot_learning_curve.py`

* :ref:`sphx_glr_auto_examples_ensemble_plot_bias_variance.py`
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* :ref:`sphx_glr_auto_examples_ensemble_plot_bias_variance.py`

This is an unrelated spot, since the example has nothing to do with learning curve.

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.

Add links to examples from the docstrings and user guide

2 participants