Skip to content

Fix R2X compatibility dependency floor - #192

Merged
pesap merged 1 commit into
mainfrom
fix/r2x-reeds-load-profile
Aug 26, 2026
Merged

pesap merged 1 commit into
mainfrom
fix/r2x-reeds-load-profile

Conversation

@pesap

@pesap pesap commented Aug 21, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Update the ReEDS R2X compatibility workflow to require r2x-reeds>=0.8.1, which contains the load-profile mapping fix for the r2x-core 0.8.0 HDF5 index behavior.

Without this floor, the R2X matrix installs the affected package and fails with polars.exceptions.DuplicateError: column 'solve_year' is duplicate while reading load.h5.

Issues resolved

  • Restores the R2X compatibility checks that currently fail on ReEDS PRs #162, #191, #170, and #157 once r2x-reeds 0.8.1 is published.

Known incompatibilities

Relevant sources or documentation

Validation, testing, and comparison report(s)

  • git diff --check
  • The affected R2X command succeeds locally with the r2x-reeds fix.

Checklist for author

Details to double-check

  • Documentation updated if necessary; no model documentation change is needed.
  • Code formatting standardized.
  • Reusable functions used where possible instead of copy/pasted code.

General information to guide review

  • Zero impact on model results; this changes only the CI dependency floor.
  • No large data files added or modified.
  • No substantive impact on runtime, folder size, process flow, or code organization.
  • No change to environment.yml or Project.toml package requirements.

Did you use LLM tools (chatbot or copilot) in the preparation of this PR?

Yes. The change and validation were prepared with an AI coding assistant and reviewed against the failing CI logs.

Tag points of contact here if you would like additional review of the relevant parts of the model

@pesap

pesap commented Aug 22, 2026

Copy link
Copy Markdown
Contributor Author

@kennedy-mindermann , these is ready for review.

@pesap
pesap force-pushed the fix/r2x-reeds-load-profile branch from 30ead77 to d0f114e Compare August 24, 2026 23:34
@pesap
pesap force-pushed the fix/r2x-reeds-load-profile branch from d0f114e to f80e05a Compare August 26, 2026 02:37
@pesap
pesap merged commit e770046 into main Aug 26, 2026
12 checks passed
@pesap
pesap deleted the fix/r2x-reeds-load-profile branch August 26, 2026 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants