Skip to content

Docs improvements#452

Draft
rgcoe wants to merge 19 commits into
sandialabs:devfrom
rgcoe:docs_april_2026
Draft

Docs improvements#452
rgcoe wants to merge 19 commits into
sandialabs:devfrom
rgcoe:docs_april_2026

Conversation

@rgcoe

@rgcoe rgcoe commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

Description

Clean up and improvements to documentation pages.

  • Add new papers and URLs for old papers to References
  • Transition references to Autograd -> Jax

TODO: not sure what this is, but maybe can be removed? https://github.com/rgcoe/WecOptTool/blob/8577c0ed17596ae47799c97c56077422f246f7ef/wecopttool/core.py#L80-L82

@rgcoe

rgcoe commented Apr 17, 2026

Copy link
Copy Markdown
Contributor Author

@dtgaebe - You could add your new animations to this PR if you like, but we (we = you and/or @jtgrasb) should first review/merge #451, as this PR take advantage of and is branched from #451.

@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 24580625441

Coverage increased (+0.1%) to 88.079%

Details

  • Coverage increased (+0.1%) from the base build.
  • Patch coverage: 5 of 5 lines across 1 file are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 1426
Covered Lines: 1256
Line Coverage: 88.08%
Coverage Strength: 5.28 hits per line

💛 - Coveralls

@jtgrasb jtgrasb self-requested a review May 14, 2026 21:13
dtgaebe added 8 commits June 18, 2026 11:20
Improve the local documentation build workflow, especially on Windows.

- make docs example copying idempotent in docs/source/conf.py
- add an environment-variable option to skip theory animation generation
- add an environment-variable option to skip gifsicle optimization
- make docs/clean_docs.py report cleanup failures without aborting

These changes help local doc builds succeed when generated files are
reused between Sphinx phases or when optional tools like gifsicle are
not installed.
Add a script and generated static assets for illustrating the
time/frequency equivalence used by the pseudo-spectral method.

This includes:
- a static figure showing the relationship between Fourier coefficients,
  harmonics, collocation points, and the periodic time trajectory
- a WaveBot animation showing an actual pseudo-spectral solution iterate history
- a Python script that reproduces these documentation assets

The generated files are committed so documentation builds do not depend on
external optimization-history data or additional runtime generation steps.
Update the theory documentation to include the new
time/frequency equivalence discussion and WaveBot solution animation.

This commit also wires the visualization generation into the
documentation build configuration.
@dtgaebe

dtgaebe commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

@rgcoe , @jtgrasb I finally added more description to the theory section with new figure and new solution animation.
I first had to change files to build the docs locally. It should make things more robust for windows. I hope I didn't break anything!

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.

3 participants