-
Notifications
You must be signed in to change notification settings - Fork 6
refactor(back_end): retire 13 uncollected self-tests behind two CI gates #113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 8 commits
Commits
Show all changes
11 commits
Select commit
Hold shift + click to select a range
ffd93ab
refactor(back_end): retire 13 uncollected self-tests behind two CI gates
guanqin-123 09dce75
refactor(back_end): retire the cons_exportor and verifier self-tests
guanqin-123 14f0e12
ci(pipeline): cover hybridz ViT verification
guanqin-123 1ccc5a7
merge: upstream/main (hybridz attention propagation)
guanqin-123 b3d3d9d
feat(back_end): generate an attention net for dual coverage
guanqin-123 c56b7e4
fix(back_end): commit generated attention fixture
guanqin-123 2567418
ci(back_end): drop the solve-path exception lint
guanqin-123 83ce9d6
fix(back_end): degenerate dot-product guard dropped the perturbation
guanqin-123 e24b95d
chore(back_end): untrack the generated attention net
guanqin-123 2a93759
ci(back_end): run the exporter sweep in the float32 job too
guanqin-123 1becb64
refactor(back_end): drop bab.py's self-tests and their CI steps
guanqin-123 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -60,11 +60,6 @@ jobs: | |
| cd ${{ github.workspace }} | ||
| coverage run -p -m act.pipeline --verify act2torch --device cpu --dtype float32 | ||
|
|
||
| - name: Run Verifier Self-Tests (float32) | ||
| run: | | ||
| cd ${{ github.workspace }} | ||
| coverage run -p -m act.back_end.verifier | ||
|
|
||
|
Comment on lines
-64
to
-67
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. be consistent with float64.yml |
||
| # ───────────────────────────────────────────────────────────────── | ||
| # Soundness check (TF-agnostic): runs once before per-solver matrix. | ||
| # See act-backend-float64.yml for the full rationale. | ||
|
|
||
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks to be auto generated for debugging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes. Now deleted.