8000 DOC Math/code formatting in docs by joshhilton · Pull Request #31325 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]

Skip to content

DOC Math/code formatting in docs #31325

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

Merged
merged 11 commits into from
May 14, 2025
Merged

Conversation

joshhilton
Copy link
Contributor
@joshhilton joshhilton commented May 6, 2025

What does this implement/fix? Explain your changes.

Many of the documentation pages were missing the proper formatting to display as inline LaTeX or code. I did not do a comprehensive check, or verify the accuracy of any of the assertions, but I fixed the formatting issues on some of the most important pages. Much more work needs to be done to clean up the documentation.

Copy link
github-actions bot commented May 6, 2025

✔️ Linting Passed

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

Generated for commit: 7cfa315. Link to the linter CI: here

@ArturoAmorQ ArturoAmorQ changed the title math/code formatting in docs DOC Math/code formatting in docs May 7, 2025
Copy link
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.

Thanks for the PR @joshhilton. Here's a batch of comments :)

joshhilton and others added 8 commits May 13, 2025 18:11
Co-authored-by: Arturo Amor <86408019+ArturoAmorQ@users.noreply.github.com>
Co-authored-by: Arturo Amor <86408019+ArturoAmorQ@users.noreply.github.com>
Co-authored-by: Arturo Amor <86408019+ArturoAmorQ@users.noreply.github.com>
Co-authored-by: Arturo Amor <86408019+ArturoAmorQ@users.noreply.github.com>
Co-authored-by: Arturo Amor <86408019+ArturoAmorQ@users.noreply.github.com>
Co-authored-by: Arturo Amor <86408019+ArturoAmorQ@users.noreply.github.com>
Co-authored-by: Arturo Amor <86408019+ArturoAmorQ@users.noreply.github.com>
@joshhilton
Copy link
Contributor Author

Thanks for the PR @joshhilton. Here's a batch of comments :)

Great, I've committed the changes. Thank you.

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

Just a couple of final tweaks but otherwise LGTM. Thanks again @joshhilton!

@ArturoAmorQ ArturoAmorQ enabled auto-merge (squash) May 14, 2025 07:55
@ArturoAmorQ ArturoAmorQ merged commit 8cfc72b into scikit-learn:main May 14, 2025
34 checks passed
@joshhilton joshhilton deleted the doc_update branch May 14, 2025 13:18
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
0