Thanks to visit codestin.com
Credit goes to github.com

Skip to content

DOC Add link to plot_lasso_coordinate_descent_path in lasso_path docstring(#30621) #31616

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 8 commits into
base: main
Choose a base branch
from

Conversation

VirenPassi
Copy link
Contributor

This PR adds a reference link to the example script
plot_lasso_coordinate_descent_path.py in the lasso_path docstring.

This is part of the documentation improvement initiative in
issue #30621 to better link example scripts to relevant API documentation.

Rendered documentation preview should now include:
"see :ref:sphx_glr_auto_examples_linear_model_plot_lasso_coordinate_descent_path.py"

Thanks!

Copy link

github-actions bot commented Jun 21, 2025

✔️ Linting Passed

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

Generated for commit: 7780d57. Link to the linter CI: here

@VirenPassi VirenPassi force-pushed the doc-add-link-plot-lasso-coordinate-descent-path-30621 branch from 1cd2eb8 to 906d6c6 Compare June 21, 2025 10:55
@VirenPassi
Copy link
Contributor Author

VirenPassi commented Jun 21, 2025

Hi! I've added a :ref: link to plot_lasso_coordinate_descent_path in the Notes section of the lasso_path docstring, following the conventions used in this file.

However, test_function_docstring[lasso_path] keeps failing. I believe this is due to pre-existing formatting issues in the docstring that my change is now surfacing.

Can a maintainer help confirm if this docstring should be patched separately or whether it's OK to ignore the failure in this PR?
(I'm happy to help fix the root docstring if needed.)

@thomasjpfan @jeremiedbb @adrinjalali

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.

1 participant