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

Skip to content

DOC using pydata doc theme - #1774

Merged
tomMoral merged 33 commits into
joblib:mainfrom
Nanored4498:new-doc-theme
Jun 8, 2026
Merged

DOC using pydata doc theme#1774
tomMoral merged 33 commits into
joblib:mainfrom
Nanored4498:new-doc-theme

Conversation

@Nanored4498

Copy link
Copy Markdown
Contributor

As discussed with @tomMoral, I tried to use the pydata theme for the sphinx doc. Let me know if it seems nice to you.

@codecov

codecov Bot commented Jan 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.21%. Comparing base (5d1653a) to head (a8e9d5e).
⚠️ Report is 27 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1774      +/-   ##
==========================================
- Coverage   94.29%   94.21%   -0.08%     
==========================================
  Files          46       46              
  Lines        7922     7922              
==========================================
- Hits         7470     7464       -6     
- Misses        452      458       +6     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@GaelVaroquaux

Copy link
Copy Markdown
Member

I had a very cursory look, but it looks cool!! Thanks

@lesteve

lesteve commented Jan 14, 2026

Copy link
Copy Markdown
Member

Maybe it's just me and old habits die hard, but looking at the rendered doc, the fact that the left-hand side navigation has disappeared feels like something is kind of missing.

Here is the left-hand side navigation I am talking about in the current doc.

image

It took me a while to find the header 😅, also you need one more click to see the different sections, first click on "User manual" then get a not so nice looking TOC, then you can chose the section you want to look at. Previously you could see the different sections directly ...

Maybe it's just me though, and also I have no idea whether this comes with pydata-sphinx-theme and there is not much we can do to tweak it.

On the plus side, probably people are more used to pydata-sphinx-theme that many projects use and having a search bar is definitely nice.

@Nanored4498

Copy link
Copy Markdown
Contributor Author

I changed a bit the looking of the TOC you referenced @lesteve . The navigation is still there but not visible from the main page. There is a navigation sidebar for each section User Manual, Module reference, Development, instead of having a global navigation sidebar.

It indeed comes from pydata-sphinx-theme, and I don't think we could tweak it. Main sections are in the header and subsections are in the sidebar. IMO it's fine, it makes the sidebar clearer, showing less information.

@lesteve

lesteve commented Jan 15, 2026

Copy link
Copy Markdown
Member

Thanks!

Maybe "Examples" are important and should be added to the top-level header as well?

skrub has it and scikit-learn too
image

@Nanored4498

Copy link
Copy Markdown
Contributor Author

Yes, I agree, will do it :)

@lesteve

lesteve commented Jan 15, 2026

Copy link
Copy Markdown
Member

I clicked around a bit and it looks good.

Two more things from me:

  • maybe use "User Guide" rather than "User Manual" since this is the term that we tend to use in scikit-learn and related projects?
  • similarly name the folder user_guide rather than user?

@tomMoral tomMoral left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A first batch of comments

Comment thread doc/_static/joblib_logo_dark.svg
Comment thread doc/user_guide/index.rst Outdated
Comment thread doc/index.rst
Comment thread .readthedocs-requirements.txt Outdated
@tomMoral tomMoral changed the title Using pydata doc theme DOC using pydata doc theme Jan 30, 2026
@tomMoral

tomMoral commented Feb 2, 2026

Copy link
Copy Markdown
Contributor

Regarding the doc, I would say:

  • Remove persistence from the front page. This is not what we want to advocate for (pickle is not great for long term persistence).
  • Add parallel backend (and reorder the user guide accordingly).
  • Add 3 extra boxes on the second row? API, examples, release notes?
  • The gray of the logo is better but can be slightly darker?

I was a bit surprised by the front page, mayeb the logo is a bit big and the purpose of the lib not so clear? Not sure how to improve though.

@Nanored4498

Copy link
Copy Markdown
Contributor Author

I tried two layouts for the first page. I would say, I prefer the logo above, but maybe you prefer inside a sidebar.
Capture d’écran du 2026-02-18 15-12-00
Capture d’écran du 2026-02-18 15-12-29

@tomMoral

Copy link
Copy Markdown
Contributor

I like both, no real preference

@tomMoral tomMoral left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It starts to look super good I like it :)

Comment thread doc/_static/versions.json Outdated
Comment thread pyproject.toml
Comment thread README.rst
Comment thread doc/user_guide/memory.rst Outdated
Comment thread doc/user_guide/memory.rst

@tomMoral tomMoral left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I find this new doc nice. Just one small question before merging.

Comment thread doc/conf.py Outdated

@tomMoral tomMoral left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this PR is mostly done.
Just 2 extra comments. also add an entry in the changelog.

Comment thread doc/conf.py Outdated
Comment thread doc/index.rst
@Nanored4498

Copy link
Copy Markdown
Contributor Author

also add an entry in the changelog.

The changelog has been added in the last commit

@tomMoral
tomMoral merged commit 6e9a70e into joblib:main Jun 8, 2026
69 of 70 checks passed
@tomMoral

tomMoral commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

thanks a lot @Nanored4498 ! This looks pretty and it is a nice re-organization :)

@Nanored4498
Nanored4498 deleted the new-doc-theme branch June 8, 2026 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants