Skip to content

DOC Add reference to plot_lasso_model_selection example in linear_model.rst(#30621)#31522

Closed
VirenPassi wants to merge 1 commit intoscikit-learn:mainfrom
VirenPassi:doc-link-lasso-model-selection
Closed

DOC Add reference to plot_lasso_model_selection example in linear_model.rst(#30621)#31522
VirenPassi wants to merge 1 commit intoscikit-learn:mainfrom
VirenPassi:doc-link-lasso-model-selection

Conversation

@VirenPassi
Copy link
Copy Markdown
Contributor

Reference Issue
#30621

What does this implement/fix?
Adds a documentation reference link for the example:
plot_lasso_model_selection.py under examples/linear_model/ to the linear_model.rst user guide.

Any additional context?
Verified that this example was not previously referenced and has now been properly linked.

@github-actions
Copy link
Copy Markdown

✔️ Linting Passed

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

Generated for commit: 5f206f5. Link to the linter CI: here

@VirenPassi VirenPassi changed the title DOC Add reference to plot_lasso_model_selection example in linear_model.rst DOC Add reference to plot_lasso_model_selection example in linear_model.rst(#30621) Jun 11, 2025
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.

Hi @VirenPassi,

thanks for opening the PR.

I have looked through where the example is already linked and found that it is linked a bit below in the part "Information-criteria based model selection" of the Lasso subsection, where it is fitting better than in the more general section, that we don't want to over crowd with another link.

I think this example is already sufficiently linked (and in the right spot too) and doesn't need to be linked further.

I will close this PR and mark it as not needing a reference.

Thank you, still for your effort. Your contribution is still helping to work towards the goal of the this meta-issue, even if it did not get merged. As written in the issue, it is expected that about 30 % of the examples, don't need to get an additional reference.

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