Repository navigation
Allow users to set metadata for BoAmps - #1234
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1234 +/- ##
==========================================
- Coverage 92.60% 92.59% -0.01%
==========================================
Files 49 49
Lines 5339 5363 +24
==========================================
+ Hits 4944 4966 +22
- Misses 395 397 +2 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
inimaz
left a comment
There was a problem hiding this comment.
Looks good @benoit-cty ! Is it ready to be merged?
bf1e207 to
398f7a6
Compare
398f7a6 to
a43a4bb
Compare
|
Pushed two commits to this branch to get CI green: |
daf0dcb to
7b022a7
Compare
Description
Allow users to set metadata for BoAmps.
Motivation and Context
BoAmps has required semantic fields (taskStage, taskFamily, algorithms, dataset) that CodeCarbon cannot infer automatically.
So users needed a way to provide BoAmps context manually.
Before this PR it was possible to do it but harder :
What changed now:
How Has This Been Tested?
With the updated example
examples/boamps_output.pyTypes of changes
What types of changes does your code introduce? Put an
xin all the boxes that apply:AI Usage Disclosure
Please refer to docs/how-to/ai-policy.md for detailed guidelines on how to disclose AI usage in your PR. Accurately completing this section is mandatory.
Checklist:
Go over all the following points, and put an
xin all the boxes that apply.