Skip to content

Export models to Ascend 910B3 - #2761

Merged
csukuangfj merged 1 commit into
k2-fsa:masterfrom
csukuangfj:ascend-910b3
Nov 10, 2025
Merged

csukuangfj merged 1 commit into
k2-fsa:masterfrom
csukuangfj:ascend-910b3

Conversation

@csukuangfj

@csukuangfj csukuangfj commented Nov 10, 2025 •

Copy link
Copy Markdown
Collaborator

Models for ascend npu can be found at
https://github.com/k2-fsa/sherpa-onnx/releases/tag/asr-models-ascend

See also modelscope/FunASR#2676 (comment)

@zqWu

Summary by CodeRabbit

  • Chores
    • Added support for Ascend 910B3 hardware platform with CANN 8.0 and 8.2 configurations
    • Extended export workflows for multiple ASR model frameworks across new platform variants
    • Updated release version tagging for improved release management

@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Nov 10, 2025
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@coderabbitai

coderabbitai Bot commented Nov 10, 2025 •

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

Walkthrough

Two Ascend NPU export workflows are updated: push trigger branches changed to "ascend-910b3", job matrices expanded with new Ascend 910B3 configurations spanning multiple model types and CANN versions (8.0, 8.2), and release tag naming updated from "asr-models" to "asr-models-ascend".

Changes

Cohort / File(s) Change Summary
Ascend 910B3 export workflow updates
.github/workflows/export-paraformer-to-ascend-npu.yaml, .github/workflows/export-sense-voice-to-ascend-npu.yaml
Updated push trigger branch to "ascend-910b3"; expanded job matrices with four new soc_version 910B3 entries each (FunASR/WSChuan-ASR and FunASR/WSYue-ASR combinations across CANN 8.0 and 8.2); updated release step tags from "asr-models" to "asr-models-ascend" for both csukuangfj and k2-fsa owners

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Both files follow identical update patterns (branch, matrix expansion, tag changes), reducing cognitive load
  • Verify matrix entries match expected Ascend 910B3 configurations and CANN versions
  • Confirm release tag changes propagate correctly across both csukuangfj and k2-fsa release steps

Possibly related PRs

Poem

🐰 A hop to Ascend's 910B3!
Four new configs in the matrix spree,
CANN 8.0 and 8.2 align,
Release tags shine ascend-redesigned,
NPU workflows now more fine! 🚀

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 049d525 and b90ba23.

📒 Files selected for processing (2)
  • .github/workflows/export-paraformer-to-ascend-npu.yaml (4 hunks)
  • .github/workflows/export-sense-voice-to-ascend-npu.yaml (4 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@csukuangfj
csukuangfj merged commit 9beac05 into k2-fsa:master Nov 10, 2025
1 of 2 checks passed
@csukuangfj
csukuangfj deleted the ascend-910b3 branch November 10, 2025 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant