Skip to content

Version dropdown#1095

Merged
xylar merged 13 commits into
MPAS-Dev:developfrom
altheaden:version-dropdown
Jun 17, 2025
Merged

Version dropdown#1095
xylar merged 13 commits into
MPAS-Dev:developfrom
altheaden:version-dropdown

Conversation

@altheaden

@altheaden altheaden commented Jun 11, 2025

Copy link
Copy Markdown
Collaborator

This PR switches the documentation workflows for MPAS-Analysis to use sphinx with @xylar's custom version switcher instead of using sphinx-multiversion.

Checklist

  • Documentation has been built locally and changes look as expected
  • Testing comment in the PR documents testing used to verify the changes

@altheaden altheaden added documenation dependencies ci Changes that affect github actions CI labels Jun 11, 2025
@xylar

xylar commented Jun 11, 2025

Copy link
Copy Markdown
Collaborator

@xylar

xylar commented Jun 11, 2025

Copy link
Copy Markdown
Collaborator

I think we're still missing changes from:
https://github.com/E3SM-Project/polaris/pull/308/files

I'm seeing docs/_templates/layout.html that is missing here. Please check if there are any other changes that didn't make the port over.

Comment thread docs/version-switcher.js
@xylar

xylar commented Jun 12, 2025

Copy link
Copy Markdown
Collaborator

@altheaden, one more place that needs updating:

cd docs
make clean
make html

@altheaden

Copy link
Copy Markdown
Collaborator Author

@xylar I think the missing files and other issues you noticed have been fixed now. Can you take another look when you get a chance?

@xylar xylar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@altheaden, we're almost there! Just a few more small changes are needed. With these, I was able to run the developer test suite and get a version of the documentation with the dropdown:
https://web.lcrc.anl.gov/public/e3sm/diagnostic_output/ac.xylar/analysis_testing/chrysalis/version-dropdown/docs/test/

Comment thread README.md Outdated
Comment thread suite/run_suite.bash Outdated
@altheaden

Copy link
Copy Markdown
Collaborator Author

@xylar I fixed those issues and located one more place where things needed updating. Does everything look okay to you?

@xylar

xylar commented Jun 17, 2025

Copy link
Copy Markdown
Collaborator

Ah, great! Thanks for finding the tutorial as well!

I was tempted to just merge but I think I should run the test suite with the latest changes to make sure everything works. I'll report back soon.

@xylar xylar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@xylar xylar merged commit 75cca38 into MPAS-Dev:develop Jun 17, 2025
5 checks passed
@altheaden altheaden deleted the version-dropdown branch June 17, 2025 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Changes that affect github actions CI dependencies documenation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants