Repository navigation
Add metadata-only Hub artifact resolution checks - #29
Conversation
Preserve existing alias and update semantics across 334 configs and 595 variants. Add ordered resolution and CPU preprocessing regression tests without changing shipped model definitions. Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
Verify exact runtime board paths using metadata only, preserve local-only restrictions and distinguish live commits from approval provenance. Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
|
@codex review Exact head Full offline CPU after integration: 1376 passed, 19 deselected. Existing all-595 variant identity proof remains green. Please review this exact integration head; dependent PRs #31/#32 have been retargeted to main before any parent branch deletion. Squash merge will occur only after exact-head agent approval. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8b64b7326c
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
🛡️ Codex Security Review · Automatically triggeredSecurity review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
Use explicit override then file_cfg target_device then Aries default, matching runtime precedence. Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
|
@codex review Exact head |
|
Codex Review: Didn't find any major issues. Nice work! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
🛡️ Codex Security Review · Automatically triggeredSecurity review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
Summary
Stack and scope
Stacked on #28 (
feat/config-inheritance-coverage); this PR adds only the Hub verifier. No existing YAML is modified. The full read-only audit found no explicit approved artifact HF commit provenance, so pinning to today'smainwould invent provenance and is intentionally deferred. Upstream timm/source-model commits are not artifact approval commits.Validation
git diff --checkpasses.Review guide
Exact head:
e8fe25c01c7ec9513485dfa164957cc3ab138a8e. Check runtime path parity, no board/root fallback for MXQ, symbolic-revision snapshot consistency, local-only no-network behavior, and propagation of API failures. Verify the network tests remain opt-in throughrequires_network.🤖 Generated with Claude Code