Fix public CI to use hf-bench uv env#155
Draft
lirundong wants to merge 2 commits into
Draft
Conversation
hannahli-nv
added a commit
that referenced
this pull request
Jun 15, 2026
Combines changes from Ocean MR !1690 and TileGym PR #155: - modeling/transformers/Dockerfile: consolidate to a single 'source' stage that copies the full repo and runs uv sync --locked --extra dev. Drops the old 'wheel' stage and system-Python pip installs. - modeling/transformers/pyproject.toml: replace pinned tilegym @ git+...@v1.3.0 with editable in-tree path (../../). - modeling/transformers/uv.lock: regenerated for in-tree tilegym. - .github/workflows/tilegym-ci.yml: update build target from 'wheel' to 'source'; run test-ops and test-benchmark via `uv run` so they use the uv venv (which has torch + in-tree tilegym), not system Python. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
hannahli-nv
added a commit
that referenced
this pull request
Jun 15, 2026
Combines changes from Ocean MR !1690 and TileGym PR #155: - modeling/transformers/Dockerfile: consolidate to a single 'source' stage that copies the full repo and runs uv sync --locked --extra dev. Drops the old 'wheel' stage and system-Python pip installs. - modeling/transformers/pyproject.toml: replace pinned tilegym @ git+...@v1.3.0 with editable in-tree path (../../). - modeling/transformers/uv.lock: regenerated for in-tree tilegym. - .github/workflows/tilegym-ci.yml: update build target from 'wheel' to 'source'; run test-ops and test-benchmark via `uv run` so they use the uv venv (which has torch + in-tree tilegym), not system Python. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
hannahli-nv
force-pushed
the
tilegym_update
branch
from
6566c06 to
53ba9aa
Compare
hannahli-nv
force-pushed
the
tilegym_update
branch
from
4fdae8c to
f3fb7f8
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Why
The public CI jobs for the modularized HF benchmark were still running tests through the container system Python. That bypassed the hf-bench uv environment and caused missing dependency failures such as PyTorch not being available.
What
sourcetarget so the source tree and uv-managed project are available.modeling/transformersviauv run --locked --no-sync python -m pytest.uv run --locked --no-sync bash ...so the benchmark subprocesses inherit the hf-bench uv environment.Assumption
This workflow change assumes the GitLab-side hf-bench Dockerfile and project metadata changes are landed into the public branch before the workflow is expected to pass end to end.
Validation
.github/workflows/tilegym-ci.ymlwith Ruby YAML..github/workflows/tilegym-ci.yml.