Skip to content

Conversation

@yiyi1991
Copy link
Contributor

@yiyi1991 yiyi1991 commented Dec 1, 2025

A draft PR to help confirm if in::methanol-final can be zero in specific regions/years.

How to review

  • @macflo8 is it possible that in::methanol-final can be zero in a regular scenario run?
  • If no, then ignore this PR.
  • If yes, read the diff, check if it is ok to replace inf with zero in the shares.
  • The CI checks all pass.

@yiyi1991 yiyi1991 self-assigned this Dec 1, 2025
@yiyi1991 yiyi1991 added report genno-based reporting and post-solve processing material MESSAGEix-Materials variant labels Dec 1, 2025
@codecov
Copy link

codecov bot commented Dec 1, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 74.9%. Comparing base (f498991) to head (67e2928).

Files with missing lines Patch % Lines
...e_ix_models/model/material/report/run_reporting.py 0.0% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##            main    #464     +/-   ##
=======================================
- Coverage   75.6%   74.9%   -0.7%     
=======================================
  Files        278     278             
  Lines      22740   22741      +1     
=======================================
- Hits       17201   17052    -149     
- Misses      5539    5689    +150     
Files with missing lines Coverage Δ
...e_ix_models/model/material/report/run_reporting.py 53.0% <0.0%> (-0.3%) ⬇️

... and 7 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

material MESSAGEix-Materials variant report genno-based reporting and post-solve processing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant