Skip to content

Update emissions accounting for trade#496

Draft
junukitashepard wants to merge 2 commits into
mainfrom
tools/bilateralize-emissions
Draft

Update emissions accounting for trade#496
junukitashepard wants to merge 2 commits into
mainfrom
tools/bilateralize-emissions

Conversation

@junukitashepard

Copy link
Copy Markdown
Contributor

CO2 emissions were not properly tracked for bilateralized trade flows through relation activity. This ensures that relation activity has proper emissions factors associated with them (negative for exports, positive for imports)

How to review

Relation activity for CO2 emissions tracking in trade should be transferred from edit to bare

PR checklist

  • Continuous integration checks all ✅
  • Add or expand tests; coverage checks both ✅
  • Add, expand, or update documentation.
  • Update doc/whatsnew.

@junukitashepard junukitashepard self-assigned this Apr 1, 2026
@junukitashepard junukitashepard added the bilateralize Issues related to the bilateralization of trade label Apr 1, 2026
@codecov

codecov Bot commented Apr 1, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 84.61538% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.3%. Comparing base (1f7fe08) to head (5609668).
⚠️ Report is 165 commits behind head on main.

Files with missing lines Patch % Lines
...ssage_ix_models/tools/bilateralize/prepare_edit.py 84.6% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##            main    #496     +/-   ##
=======================================
- Coverage   73.3%   73.3%   -0.1%     
=======================================
  Files        301     301             
  Lines      24429   24440     +11     
=======================================
+ Hits       17914   17922      +8     
- Misses      6515    6518      +3     
Files with missing lines Coverage Δ
...ssage_ix_models/tools/bilateralize/prepare_edit.py 89.5% <84.6%> (-0.3%) ⬇️

... and 1 file 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

bilateralize Issues related to the bilateralization of trade

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant