Skip to content

DOC Add dropdowns to module 1.1 Linear Models#26623

Merged
ArturoAmorQ merged 16 commits intoscikit-learn:mainfrom
Tech-Netiums:sphinx_dropdown_linear_models
Oct 11, 2023
Merged

DOC Add dropdowns to module 1.1 Linear Models#26623
ArturoAmorQ merged 16 commits intoscikit-learn:mainfrom
Tech-Netiums:sphinx_dropdown_linear_models

Conversation

@Tech-Netiums
Copy link
Copy Markdown
Contributor

@Tech-Netiums Tech-Netiums commented Jun 19, 2023

Add dropdowns to submodule 1.1 Linear models regarding #26617

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 19, 2023

✔️ Linting Passed

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

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

@Tech-Netiums Tech-Netiums changed the title Add dropdowns to module 1.1 Linear Models DOC Add dropdowns to module 1.1 Linear Models Jun 20, 2023
Copy link
Copy Markdown
Member

@GaelVaroquaux GaelVaroquaux left a comment

Choose a reason for hiding this comment

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

I think that this is super useful!!

One tiny comment: I think that we should also fold 1.1.8.1

And +1 for merge once this is implemented

@glemaitre glemaitre self-requested a review June 22, 2023 12:38
Copy link
Copy Markdown
Member

@glemaitre glemaitre left a comment

Choose a reason for hiding this comment

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

Here are some comments. I think that we can further improve the page but I would prefer to make smaller PR dedicated to each section to make it easy to review.

Copy link
Copy Markdown
Member

@ArturoAmorQ ArturoAmorQ left a comment

Choose a reason for hiding this comment

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

Can you please solve the conflicts with main, @Tech-Netiums ? Otherwise LGTM :)

Copy link
Copy Markdown
Member

@ArturoAmorQ ArturoAmorQ left a comment

Choose a reason for hiding this comment

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

I will take the liberty of pushing some commits to your PR to avoid breaking sphinx permalinks.

@ArturoAmorQ
Copy link
Copy Markdown
Member

Thanks for your time and effort @Tech-Netiums, merging :)

@ArturoAmorQ ArturoAmorQ merged commit fa84c90 into scikit-learn:main Oct 11, 2023
Copy link
Copy Markdown

@KartikeyBartwal KartikeyBartwal left a comment

Choose a reason for hiding this comment

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

I gained quite a bit of clarity from your PR 🙌

glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Oct 31, 2023
REDVM pushed a commit to REDVM/scikit-learn that referenced this pull request Nov 16, 2023
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.

5 participants