Skip to content

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

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

Conversation

VirenPassi
Copy link
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.

Copy link

✔️ 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
Contributor

@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