Compute emissions based on ongoing carbon intensity

[cianc]

Description

Compute emissions based on ongoing carbon intensity samples, rather than just taking the latest value and applying it to all energy consumption every time.

Related Issue

Please link to the issue this PR resolves: #994

Motivation and Context

Improves emission accuracy over long runs.

How Has This Been Tested?

Added a new test and ran all existing tests.

Screenshots (if appropriate):

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

Go over all the following points, and put an x in all the boxes that apply.

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING.md document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.73%. Comparing base (abcaf04) to head (cc62d32).
⚠️ Report is 6 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1050      +/-   ##
==========================================
+ Coverage   68.51%   68.73%   +0.21%     
==========================================
  Files          41       41              
  Lines        3862     3873      +11     
==========================================
+ Hits         2646     2662      +16     
+ Misses       1216     1211       -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[benoit-cty]

Thank you !
Because some people are using CodeCarbon with short tasks, the potential problem I see is the time it will take to do it. Maybe it will be better to have something like a cache with a 5 minutes duration, what do you think ?

[cianc]

Thank you ! Because some people are using CodeCarbon with short tasks, the potential problem I see is the time it will take to do it. Maybe it will be better to have something like a cache with a 5 minutes duration, what do you think ?

Could you elaborate on this, I think I'm missing something? Are you referring to the time it takes to call electricitymaps_api.get_emissions? I don't think this PR changes the number of these calls, it just changes the accounting.

[benoit-cty]

Sorry, you are absolutely right, the def _update_emissions is just calling the same function as before.

Perf improvement is another subject.