Skip to content

NPU add titan test #55

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 10 commits into from
Jun 3, 2025
Merged

NPU add titan test #55

merged 10 commits into from
Jun 3, 2025

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.

2 participants