Repository navigation
Conversation
|
Hello, thanks for the contribution. Can you keep our PR template ? |
060a21e to
a3ede3f
Compare
|
Updated the PR description to the complete repository template, including the required 🟠 AI-generated disclosure. I personally reviewed and tested every submitted line, understand the implementation, and can explain and support it through review and after merge. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1275 +/- ##
=======================================
Coverage 92.07% 92.08%
=======================================
Files 49 49
Lines 5202 5204 +2
=======================================
+ Hits 4790 4792 +2
Misses 412 412 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Assisted-by: OpenAI Codex GPT-5 <noreply@openai.com> Signed-off-by: Loi Nguyen <vanloi999@gmail.com>
a3ede3f to
5b4f3c1
Compare
|
At exact head
Public CLI flags also need docs ( PR body is stale: it still says option names are derived from Also rebase onto current |
Keep master's config-aware defaults for project_name, measure_power_secs, api_call_interval and log_level, and add the explicit tracker options on top. The forwarding test now stubs the user config and no longer expects region when it is not given.
Description
Forward tracker constructor options passed to
codecarbon monitor, including--pueand--allow-multiple-runs. Accepted option names are derived fromBaseEmissionsTracker.__init__so the CLI continues to recognize future scalar tracker options. CLI coverage verifies typed float and boolean forwarding.Related Issue
Fixes #1273
Motivation and Context
The monitor command accepted tracker configuration but did not forward several supported constructor options. This made CLI monitoring behave differently from direct tracker use.
How Has This Been Tested?
uv run python -m pytest tests/cli/test_cli_main.py -quv run ruff check codecarbon/cli/main.py tests/cli/test_cli_main.pyuv run ruff format --check codecarbon/cli/main.py tests/cli/test_cli_main.pyuv run mypy codecarbon/cli/main.pyis currently blocked by 285 existing type errors across 19 project modules.Screenshots (if appropriate):
Not applicable.
Types of changes
AI Usage Disclosure
OpenAI Codex assisted with implementation and test preparation. I personally reviewed and tested every submitted line, understand the implementation, and can explain and maintain it.
Checklist: