Switch from Hatch to UV #815
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
benoit-cty
changed the title
benoit-cty
force-pushed
the
codecarbon_v3_rc
branch
from
8eac358 to
0792bc8
Compare
benoit-cty
force-pushed
the
feat/switch_to_uv
branch
from
dd2e791 to
43e2c71
Compare
added 2 commits
July 2, 2025 12:38
benoit-cty
changed the title
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR replaces Hatch with UV and Taskipy for environment and task management, updates documentation accordingly, and refines the CPU power estimation logic.
- Migrate project scripts and environments from Hatch to UV/Taskipy in pyproject.toml
- Update CONTRIBUTING.md to use UV commands instead of Hatch
- Change CPU load–based power calculation and debug logging in hardware.py
Reviewed Changes
Copilot reviewed 3 out of 5 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| pyproject.toml | Switched [tool.hatch] and optional-dependencies to UV’s dependency-groups and Taskipy tasks |
| codecarbon/external/hardware.py | Refactored CPU load measurement and power estimation, adjusted debug messages |
| CONTRIBUTING.md | Updated setup and command instructions from Hatch to UV |
Comments suppressed due to low confidence (3)
pyproject.toml:127
- [nitpick] Task names mix hyphens (
pre-commit-install) and no hyphens (precommit-install). Consider standardizing on one naming convention to improve clarity.
precommit-install = "pre-commit install"
codecarbon/external/hardware.py:215
- This comment is misleading—
cpu_loadis still in 0–100% range here. Either remove the comment or explicitly divide by 100 when assigning.
) # Convert to 0-1 range
codecarbon/external/hardware.py:221
- [nitpick] Using the debug spec
{self._tdp=}may clutter logs with the variable name. Consider formatting as{self._tdp}and logging load_factor separately for readability.
f"CPU load {self._tdp} W and {cpu_load:.1f}% {load_factor=} => estimation of {power} W for whole machine."
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
added 3 commits
July 6, 2025 17:24
benoit-cty
force-pushed
the
feat/switch_to_uv
branch
from
43ca859 to
7f3c3ce
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The goal of this PR is to use UV + Taskipy instead of Hatch.
TODO:
Maybe we could go with UV and Hatch for a first release and remove Hatch after some time ?