Skip to content

Commit 3898567

Browse files
authored
Drop Levanter protobuf upper bound (#5171)
## Summary - Drop Levanter's direct `protobuf<7` upper bound now that TensorBoard/XProf compatibility has been rechecked. - Refresh the lockfile for the affected profiling packages. Fixes #5139. ## Validation - `uv lock -P protobuf -P tensorboard -P tensorboardx -P xprof` - `uv run --package marin-levanter --extra profiling --group test pytest lib/levanter/tests/test_tensorboard.py` - `uv run --isolated --with 'protobuf>=7' --with 'tensorboard>=2.16' --with 'tensorboardX>=2.6' --with 'xprof' python -c "..."` - `./infra/pre-commit.py --fix lib/levanter/pyproject.toml uv.lock` - `uv lock --check` - `git diff --check` Co-authored-by: Romain Yon <1596570+yonromai@users.noreply.github.com>
1 parent 28163ea commit 3898567

2 files changed

Lines changed: 19 additions & 19 deletions

File tree

lib/levanter/pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ dependencies = [
7070
"deepdiff>=8.6.2",
7171
"lenses",
7272
"jinja2", # needed for apply_chat_template
73-
"protobuf>=6,<7", # ensure TB/XProf compatibility
73+
"protobuf>=6",
7474
"immutabledict",
7575
]
7676

uv.lock

Lines changed: 18 additions & 18 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)