Skip to content

Conversation

@forsyth2
Copy link
Collaborator

Part of the implementation for E3SM-Project/zppy#697

@forsyth2 forsyth2 self-assigned this Mar 26, 2025
forsyth2 and others added 2 commits March 28, 2025 11:58
- Add original_plots_multi flag to Parameters class
- Modify make_plot_pdfs to use multi-plot layout for original plots
- Change coupled_global to exclude original plots from interactive viewer
- Add special handling for original plots in index page

Now when make_viewer=True:
- Regular component plots use interactive viewer
- Original plots are always displayed as multi-plot PDFs
- Index page includes links to both interactive viewers and PDF files

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@forsyth2 forsyth2 force-pushed the fix_plots_lnd branch 2 times, most recently from 6cbb4fb to 5e2894e Compare March 28, 2025 18:52
@forsyth2 forsyth2 marked this pull request as ready for review March 31, 2025 17:54
@forsyth2
Copy link
Collaborator Author

Approved via E3SM-Project/zppy#697 (review)

@forsyth2 forsyth2 changed the title Fix plots_lnd Improve variable and viewer handling Mar 31, 2025
@forsyth2 forsyth2 merged commit 460a87f into main Mar 31, 2025
4 checks passed
@forsyth2 forsyth2 deleted the fix_plots_lnd branch March 31, 2025 18:00
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