Repository navigation
knowledge: add PyTorch and TorchNPU skills without process artifacts - #312
Merged
Merged
Conversation
Add two source-grounded operating graphs with eight child procedures each, retained source provenance and notices, and focused reader and packaging coverage. Keep construction inventories and process reports in their immutable historical delivery instead of the maintained source tree. Agent-assisted-by: Codex / gpt-6.1-sol [effort=xhigh] (PyTorch source author) Agent-assisted-by: Codex / gpt-6.1-sol [effort=xhigh] (TorchNPU source author) Agent-assisted-by: Codex / gpt-6.1-sol [effort=xhigh] (integration author) Signed-off-by: David Chen <530634352@qq.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. |
Skill evaluationPLANNED: not executed; there are no task outcomes yet.
Author verification steps (nonpaid; not yet run)
Evidence and logs · Retention: 30 days. Plan bindings, budget, frozen exam and advisory detailsAdvisory: obtain missing evidenceRetained evidence availability: planned / unexecuted. No approval or merge authorization is issued.
Model/settings:
Automatic execution / cumulative PR budget: {"reason":"Trusted configuration or pricing missing, invalid or expired","remaining_usd":"unknown","reserved_or_unresolved_usd":"unknown","spent_or_legacy_usd":"unknown","state":"blocked"}Projection check selection (not an executed result): {"reason":"No verified ancestor source pin for automatic command selection","status":"unavailable"}Provider/settings: Uncovered / unsupported paths (206)
|
Collaborator
Author
|
Independent reviewer
The reviewer did not rerun the broader focused suite, wheel installation or device tests. Those checks are separately attributed; historical native/accelerator limitations remain unchanged. Parent transcribed the independent result. |
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.
Refs #309
Reintroduces the PyTorch and TorchNPU operating skills after the owner-requested revert #311, as one signed-off commit based on
977289d2.The replacement contains 206 files, +9,352 / −277 lines, compared with the original 366 files, +229,535 / −277: approximately 96% fewer additions. It omits all 160 construction-inventory and process-report files. Historical evidence remains linked at the immutable original delivery instead of being copied into the maintained tree.
Retained scope:
Validation so far: all 681 generator input paths and raw-byte hashes exactly match the published source revision
8115da15b9ef34978de500b08aa9f14ea790c35b, so the existing generated catalog remains accurate. Full-tree Ruff and whitespace checks (withcore.whitespace=cr-at-eolfor byte-preserved upstream notices) passed.Final validation at
e3247382860bae094d7ca81e164ba2d28bd3e84b:An initial local run recorded 121 passed / 1 failed while it overlapped the parent's rebase checkout; the failure observed a transiently absent skill root. The stable reruns required no code fix. That interrupted-content run is retained as a test-coordination failure, not counted as passing evidence.
The historical source/usability evidence applies to unchanged runtime bytes; this repackaging does not rerun or upgrade it. CUDA/NPU/HCCL and pinned-source builds remain unverified; Windows Gloo failures remain recorded historically. Native execution is not claimed.