Skip to content

[ci] 添加 nightly build workflow#129

Open
kerer-ai wants to merge 10 commits into
Ascend:masterfrom
kerer-ai:master_nightly_build
Open

[ci] 添加 nightly build workflow#129
kerer-ai wants to merge 10 commits into
Ascend:masterfrom
kerer-ai:master_nightly_build

Conversation

@kerer-ai
Copy link
Copy Markdown
Collaborator

@kerer-ai kerer-ai commented May 6, 2026

  • Refactor _build.yml: fix git proxy redundancy, remove duplicate pip upgrades, add warning for CANN env failures, use $HOME instead of hardcoded paths
  • Add _test.yml: reusable test workflow with NPU verification and smoke test
  • Add nightly.yml: main workflow with schedule, PR trigger, and manual dispatch

- Refactor _build.yml: fix git proxy redundancy, remove duplicate pip upgrades,
  add warning for CANN env failures, use $HOME instead of hardcoded paths
- Add _test.yml: reusable test workflow with NPU verification and smoke test
- Add nightly.yml: main workflow with schedule, PR trigger, and manual dispatch

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ascend-robot
Copy link
Copy Markdown

CLA Signature Pass

kerer-ai, thanks for your pull request. All authors of the commits have signed the CLA. 👍

- Add PIP_CACHE_DIR and CCACHE_DIR env variables
- Replace $HOME references with ${{ env.* }} for proper variable expansion
- actions/cache does not expand shell variables in 'with' parameters

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ascend-robot
Copy link
Copy Markdown

CLA Signature Pass

kerer-ai, thanks for your pull request. All authors of the commits have signed the CLA. 👍

Changes:
- Rename docker_image_tag to docker_image, accept full URL instead of tag
- Set default image: quay.io/kerer/pytorch:manylinux-cann9.0.0-beta.2-20260428
- Remove docker image拼接 in _build.yml and _test.yml
- Unify error handling in _test.yml (use ::warning:: instead of || true)
- Remove unused DOCKER_REGISTRY env in nightly.yml

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ascend-robot
Copy link
Copy Markdown

CLA Signature Pass

kerer-ai, thanks for your pull request. All authors of the commits have signed the CLA. 👍

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ascend-robot
Copy link
Copy Markdown

CLA Signature Pass

kerer-ai, thanks for your pull request. All authors of the commits have signed the CLA. 👍

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ascend-robot
Copy link
Copy Markdown

CLA Signature Pass

kerer-ai, thanks for your pull request. All authors of the commits have signed the CLA. 👍

- Change if: always() to if: success() or skipped with skip_build
- Remove Check overall status step (GitHub handles failure status automatically)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ascend-robot
Copy link
Copy Markdown

CLA Signature Pass

kerer-ai, thanks for your pull request. All authors of the commits have signed the CLA. 👍

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ascend-robot
Copy link
Copy Markdown

CLA Signature Pass

kerer-ai, thanks for your pull request. All authors of the commits have signed the CLA. 👍

Changed std::shared_ptr<Node> to c10::intrusive_ptr<Node> in unpack_list
and unpack_opt_list functions to match PyTorch's new autograd API.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ascend-robot
Copy link
Copy Markdown

CLA Signature Pass

kerer-ai, thanks for your pull request. All authors of the commits have signed the CLA. 👍

…eFallbackKernel

Changed WarnNotImplemented from std::shared_ptr to c10::intrusive_ptr to match
PyTorch's autograd API changes:
- Use c10::make_intrusive instead of new with deleteNode
- Remove deprecated torch::autograd::deleteNode usage

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ascend-robot
Copy link
Copy Markdown

CLA Signature Pass

kerer-ai, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@ascend-robot
Copy link
Copy Markdown

CLA Signature Pass

kerer-ai, thanks for your pull request. All authors of the commits have signed the CLA. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants