Skip to content

Nash Convergence Report Enhancements - #802

Draft
Renato-Rodrigues wants to merge 19 commits into
pik-piam:masterfrom
Renato-Rodrigues:nash-convergence-report
Draft

Nash Convergence Report Enhancements#802
Renato-Rodrigues wants to merge 19 commits into
pik-piam:masterfrom
Renato-Rodrigues:nash-convergence-report

Conversation

@Renato-Rodrigues

@Renato-Rodrigues Renato-Rodrigues commented Jun 5, 2026

Copy link
Copy Markdown
Member

Purpose of this PR

This PR introduces add enhancements to the Nash Convergence Report, improving readability, analysis depth, and navigation. It also includes several bug fixes and maintenance improvements for package compliance.

  • New Summary Tab: Added a high-level summary tab featuring aggregated activity and convergence timelines, a final deviation vs. tolerance chart, and a comprehensive summary table built with reactable.

  • Collapsible Sections: Reorganized the region-specific tabs to use collapsible sections (<details> style), reducing visual clutter and improving navigation across large reports.

  • Expanded Chart Suite: Added several new visualizations per target group, including:

    • Rescale Factor evolution
    • Phase Diagrams
    • Slope and Clamp Evolution
    • Convergence Rate tracking
  • Rescale Logic Table: Integrated a detailed table for debugging regipol targets, explicitly showing the active rescale methods per iteration.

  • UI/UX Improvements:

    • Updated the color palette: converged iterations now use a distinct light green (#93c47d), and sl-ref uses a new bluish color.
    • Enhanced tooltips to show additional data like slope values.
    • Disabled Plotly hover menus on charts where they are unnecessary to reduce interface clutter.
  • Other changes: Increased the timeout for download.file to 300 seconds in test-convGDX2mif.R to prevent test failures on slow connections when fetching large test GDX files.

You can see an examples at: /p/projects/ecemf/REMIND/debug/remind_leastSquareSlope/output

Checklist:

I checked the tests when running buildLibrary and made sure that my changes

  • do not create new complaints about summation checks.
  • do not create new complaints about missing variables that are expected in the piamInterfaces package. (If needed, adjust piamInterfaces mappings based on the README.md. In case of complaints unrelated to your changes that you are unable to fix, please open an issue in piamInterfaces.)

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.

1 participant