Skip to content

Quick check in for inf numbers in the share data in material reporting#464

Draft
yiyi1991 wants to merge 1 commit intomainfrom
material_report_check
Draft

Quick check in for inf numbers in the share data in material reporting#464
yiyi1991 wants to merge 1 commit intomainfrom
material_report_check

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 70.3%. Comparing base (2d6ad91) to head (4da073a).

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   71.0%   70.3%   -0.7%     
=======================================
  Files        286     286             
  Lines      23106   23107      +1     
=======================================
- Hits       16414   16265    -149     
- Misses      6692    6842    +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