Skip to content

Conversation

@lowdy1
Copy link
Collaborator

@lowdy1 lowdy1 commented Apr 28, 2025

No description provided.

uses: ./.github/workflows/_ascend_npu_torchtitan.yml
with:
runner: "linux-arm64-npu-2"
image: ${{ needs.prepare.outputs.image }}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use ascendai/cann:8.0.0.beta1-910b-ubuntu22.04-py3.10 temporarily

Comment on lines -154 to -138
benchmark:
name: Run benchmarks
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why remove this?


jobs:
setup_environment:
name: Run Torchtitan Environment
Copy link
Contributor

@shink shink May 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
name: Run Torchtitan Environment
name: run torchtitan tests

Comment on lines 115 to 117
- name: Install nightly torchvision and torchaudio
run: |
pip install --pre torchvision torchaudio --no-deps --index-url https://download.pytorch.org/whl/nightly/cpu
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no need to install these two

Comment on lines 132 to 137
- name: Run torchtitan integration_test
working-directory: torchtitan
run: |
mkdir artifacts-to-be-uploaded
npu_count=$(python -c "import torch; print(torch.npu.device_count())")
python ./tests/integration_tests.py artifacts-to-be-uploaded --ngpu ${npu_count}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good. And should we upload this dir artifacts-to-be-uploaded?

@shink
Copy link
Contributor

shink commented May 30, 2025

Please rebase your branch after #60 landed

@lowdy1 lowdy1 merged commit a2ba986 into main Jun 3, 2025
lowdy1 added a commit that referenced this pull request Jun 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants