Skip to content
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

More docs unification #79

Merged
merged 3 commits into from
Aug 15, 2022
Merged

More docs unification #79

merged 3 commits into from
Aug 15, 2022

Conversation

ArnoStrouwen
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Aug 2, 2022

Codecov Report

Merging #79 (7243315) into master (b225826) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #79   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files           8       8           
  Lines         466     466           
======================================
  Misses        466     466           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

docs/src/index.md Outdated Show resolved Hide resolved
docs/pages.jl Outdated
@@ -1,7 +1,7 @@
# Put in a separate page so it can be used by SciMLDocs.jl

pages = [
"Home" => "index.md",
"Overview" => "index.md",
Copy link
Member

Choose a reason for hiding this comment

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

We do need some kind of tag line.

Screenshot 2022-08-02 094516

Otherwise it won't do well in search. This doesn't necessarily need to be the label in the SciMLDocs.

Copy link
Member Author

Choose a reason for hiding this comment

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

I don't immediately see a way for the two to be separated?
The <title> element of https://nonlinearsolve.sciml.ai/ seems to be "first item on dropdown menu · NonLinearSolve.jl"
where NonLinearSolve.jl is what is used as the sitename keyword argument in the makedocs function.

Copy link
Member

Choose a reason for hiding this comment

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

We can make a bool that is set before pages.jl is loaded that then allowed for is_scimldocs and to switch whether to place the title on or not.

But I would greatly prefer just always having the title so it shows up on search engines, and making an issue to Documenter.jl to decouple the sidebar name from the title.

Copy link
Member Author

Choose a reason for hiding this comment

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

You can give a different sidebar name using that technique, but you still have to fill in something for the sidebar for is_scimldocs=false.

Currently for DifferentialEquations.jl it is

<title>DifferentialEquations.jl: Scientific Machine Learning (SciML) Enabled Simulation and Estimation · DifferentialEquations.jl</title>

Which has DifferentialEquations in there twice, once from the a name in the dropdown menu and once from the sitename keyword argument.

So you could make the sitename argument an empty string. But then if you go to down the dropdown menu e.g

<title>Solving Large Stiff Equations · DifferentialEquations.jl</title>

It would also lose the DifferentialEquations, which is not great if somebody is searching for: "stiff equations DifferentialEquations.jl".

I'll make an issue with Documenter.jl .

@ArnoStrouwen
Copy link
Member Author

The way it is done now should give:

<title>NonlinearSolve.jl: High-Performance Unified Nonlinear Solvers · SciML</title> which is in my opinion the prettiest option given Documenter's current limitations. Dropdown menu to be fixed later.

@ChrisRackauckas ChrisRackauckas merged commit a20dd5d into SciML:master Aug 15, 2022
avik-pal pushed a commit that referenced this pull request Nov 1, 2024
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.

2 participants