Skip to content

feat: add BoAmps as output method#1113

Open
SaboniAmine wants to merge 1 commit intomasterfrom
feat/add_boamps_as_output_method
Open

feat: add BoAmps as output method#1113
SaboniAmine wants to merge 1 commit intomasterfrom
feat/add_boamps_as_output_method

Conversation

@SaboniAmine
Copy link
Member

Description

This PR introduces a new output method, producing files respecting the BoAmps export format standard.

Related Issue

No issue related.

Motivation and Context

Why is this change required? What problem does it solve?

How Has This Been Tested?

Please describe in detail how you tested your changes.

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.

@SaboniAmine SaboniAmine requested a review from a team as a code owner March 17, 2026 12:40
@SaboniAmine SaboniAmine force-pushed the feat/add_boamps_as_output_method branch from b5ea5f5 to 1275d5c Compare March 17, 2026 12:41
@codecov
Copy link

codecov bot commented Mar 17, 2026

Codecov Report

❌ Patch coverage is 98.80952% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.14%. Comparing base (6dbd125) to head (cb498c5).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
codecarbon/output_methods/boamps/models.py 99.10% 2 Missing ⚠️
codecarbon/output_methods/boamps/output.py 96.15% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1113      +/-   ##
==========================================
+ Coverage   80.73%   82.14%   +1.40%     
==========================================
  Files          41       45       +4     
  Lines        3920     4256     +336     
==========================================
+ Hits         3165     3496     +331     
- Misses        755      760       +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 benoit-cty force-pushed the feat/add_boamps_as_output_method branch from 1275d5c to cb498c5 Compare March 18, 2026 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant