DOC Add link to plot_lasso_and_elasticnet.py example in linear model #30587
Conversation
…lasso and elastic net
…lasso and elastic net
…lasso and elastic net patched
There was a problem hiding this comment.
Thank you for the PR @marouaneaka! I have a few comments:
- It might be helpful to include a link to the example in the ARD regression documentation as well.
- In the PR description, please update the text to "Towards 26927" to avoid automatically closing the meta-issue when the PR is merged.
| @@ -875,6 +875,10 @@ class ElasticNet(MultiOutputMixin, RegressorMixin, LinearModel): | |||
| 1.451... | |||
| >>> print(regr.predict([[0, 0]])) | |||
| [1.451...] | |||
|
|
|||
| For an example showcasing Elastic-Net alongside Lasso and ARDRegression for sparse | |||
There was a problem hiding this comment.
"ARD regression", and likewise for Lasso.
|
Hi thanks for the comments : - Added link in ARD Regression, fixed typo mistake in elastic net and lasso. if there is anything else just let me know :) |
There was a problem hiding this comment.
Thanks for updating the PR, @marouaneaka! I have a few more comments.
In the ARD Regression documentation, could you please move the following example:
down to where your L1 example is? After that, you can delete the "Notes" section.
|
Just updated ARD documentation as specified |
There was a problem hiding this comment.
Thank you for updating the PR, @marouaneaka! Could you address the CI failure? If you click on "Details" under "scikit-learn.scikit-learn," you’ll find the error message(s).
|
Hi @marouaneaka! Would you like to continue working on this PR? Let us know if you need help. |
Reference Issues/PRs
What does this implement/fix? Explain your changes.
Towards 26927
Added links to lasso and elasticnet classes
Any other comments?