Skip to content

Export models for CANN toolkit 7.0 - #2795

Merged
csukuangfj merged 1 commit into
k2-fsa:masterfrom
csukuangfj:cann-7.0
Nov 18, 2025
Merged

csukuangfj merged 1 commit into
k2-fsa:masterfrom
csukuangfj:cann-7.0

Conversation

@csukuangfj

@csukuangfj csukuangfj commented Nov 18, 2025 •

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Chores
    • Updated build workflow to support Cann 7.0 across multiple hardware platforms (910B, 910B2, 910B3, 310P3).
    • Added environment configurations for FunASR and WSYue-ASR frameworks with new version compatibility.

@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.

@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Nov 18, 2025
@csukuangfj
csukuangfj merged commit 3f13f7a into k2-fsa:master Nov 18, 2025
1 check was pending
@coderabbitai

coderabbitai Bot commented Nov 18, 2025 •

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

Walkthrough

This PR updates the Ascend NPU export GitHub Actions workflow to support Cann 7.0 by expanding the strategy matrix with Cann 7.0.0/7.0 configurations across four SOC versions (910B, 910B2, 910B3, 310P3) and adding corresponding LD_LIBRARY_PATH environment variables for the Cann 7.0.0 development library directory.

Changes

Cohort / File(s) Change Summary
Ascend Cann 7.0 Support
\.github/workflows/export-sense-voice-to-ascend-npu\.yaml
Updated branch filter from ascend-910b3 to cann-7.0; expanded strategy matrix with Cann 7.0.0/7.0 configurations for FunASR and WSYue-ASR across 910B, 910B2, 910B3, and 310P3 SOC versions; added LD_LIBRARY_PATH entries pointing to /usr/local/Ascend/ascend-toolkit/latest/x86_64-linux/devlib/x86_64 in environment verification and model export steps.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • The changes follow a repetitive pattern: identical Cann 7.0 configurations applied across four SOC versions with consistent LD_LIBRARY_PATH additions.
  • Primary focus areas: verify the correctness of the LD_LIBRARY_PATH for Cann 7.0.0, ensure all four SOC versions have identical Cann 7.0 entries, and confirm branch filter change doesn't break existing logic.

Possibly related PRs

Suggested labels

size:M

Poem

🐰 A hop through Cann's new path so bright,
Seven-oh flows through libraries of light,
Ascend's four SOCs now in a row,
FunASR and WSYue ready to go,
Hop-hop! Export workflows take flight! 🚀

✨ 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 6659d47 and ec3063b.

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

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

  • Provide your own instructions using the high_level_summary_instructions setting.
  • Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
  • Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

  1. 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
  2. 📓 References — List relevant issues, discussions, documentation, or related PRs.
  3. 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
  4. 📊 Contributor Summary — Include a Markdown table showing contributions:
    | Contributor | Lines Added | Lines Removed | Files Changed |
  5. ✔️ Additional Notes — Add any extra reviewer context.
    Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.


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.

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