update infra/ci tools and bump CT sha#171
Merged
speediedan merged 3 commits intomainfrom Sep 18, 2025
Merged
Conversation
…iform, upgrade circuit-tracer sha and flip default to use forthcoming pypi installation, make post_upgrades default to false
github-actions
bot
added
area: tests
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR standardizes infrastructure and CI tool options, updates circuit-tracer integration, and reorganizes requirements file structure. The main purpose is to make command-line flags uniform across scripts, update circuit-tracer to a new commit SHA, and move CI-specific requirements to a consolidated location.
- Unifies command-line argument format from underscores to hyphens (e.g.,
--repo_hometo--repo-home) - Updates circuit-tracer to new commit SHA
8941776fdbe6f37b4c87e5bb2c14b8b746be0f9band changes default installation mode - Consolidates CI requirements files under
requirements/ci/directory structure
Reviewed Changes
Copilot reviewed 28 out of 30 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/core/test_regen_reqfiles.py | Updates test file paths to use consolidated CI requirements directory |
| src/interpretune/tools/install_circuit_tracer.py | Updates circuit-tracer SHA, changes pin file location, and reverses CLI flag logic |
| scripts/*.sh | Standardizes CLI arguments from underscore to hyphen format across all shell scripts |
| requirements/regen_reqfiles.py | Updates file paths to use CI directory for post_upgrades and circuit_tracer_pin |
| requirements/ files | Moves and consolidates CI-specific requirements under requirements/ci/ |
| .github/ workflows and actions | Updates file paths to reference new CI requirements location |
| pyproject.toml | Updates datasets version requirement and removes post_upgrades configuration |
| docs/ and configuration files | Updates references to new file locations and CLI argument formats |
|
No CI pin file changes detected by regen-ci-req-check. |
speediedan
force-pushed
the
update_ct_package_tools
branch
from
1634de6 to
f66e9d6
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
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.
cleanup ci-specific requirements files, make infra/ci tool options uniform, upgrade circuit-tracer sha and flip default to use forthcoming pypi installation, make post_upgrades default to false