Skip to content

Export SenseVoice models for Ascend910B2 NPU - #2740

Merged
csukuangfj merged 1 commit into
k2-fsa:masterfrom
csukuangfj:export-ascend-910B2
Nov 2, 2025
Merged

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

Conversation

@csukuangfj

@csukuangfj csukuangfj commented Nov 2, 2025 •

Copy link
Copy Markdown
Collaborator

See also
modelscope/FunASR#2676 (comment)

To fix the following error:

root@910B-36:/sense-voice-910B# python test_om.py 
[INFO] acl init success
[INFO] open device 0 success
[INFO] create new context
[ACL ERROR] EE1001: [PID: 106] 2025-10-30-08:21:08.817.477 The argument is invalid.Reason: ModelCheckCompatibility failed, supported socVersion=Ascend910B2, but the model socVersion=Ascend910B
        Solution: 1.Check the input parameter range of the function. 2.Check the function invocation relationship.
        TraceBack (most recent call last):
        rtModelCheckCompatibility execute failed, reason=[soc version error][FUNC:FuncErrorReason][FILE:error_message_manage.cc][LINE:53]
        Call rtModelCheckCompatibility(model_soc_version.c_str(), model_arch_type.c_str()) fail, ret: 0x7BC80[FUNC:CheckModelCompatibility][FILE:om_file_helper.cc][LINE:265]
        Check model compatibility failed.[FUNC:LoadModelData][FILE:model_helper.cc][LINE:1266]
        Assert ((GenerateGeRootModel(om_load_helper, model_data)) == ge::SUCCESS) failed[FUNC:LoadRootModel][FILE:model_helper.cc][LINE:1116]
        Assert ((error_code) == ge::SUCCESS) failed[FUNC:LoadExecutorFromModelData][FILE:api.cc][LINE:123]
        [Model][FromData]call gert::LoadExecutorFromModelDataWithMem load model from data failed, ge result[1343225857][FUNC:ReportCallError][FILE:log_inner.cpp][LINE:162]

[ERROR] load model from file failed, model file is ./model.om
[WARN] Check failed:processModel->LoadModelFromFile(modelPath), ret:1

[WARN] no model had been loaded, unload failed
Traceback (most recent call last):
  File "/sense-voice-910B/test_om.py", line 148, in <module>
    main()
  File "/sense-voice-910B/test_om.py", line 95, in main
    model = OmModel()
            ^^^^^^^^^
  File "/sense-voice-910B/test_om.py", line 16, in __init__
    self.model = InferSession(device_id=0, model_path="./model.om", debug=False)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/python3.11.13/lib/python3.11/site-packages/ais_bench/infer/interface.py", line 82, in __init__
    self.session = aclruntime.InferenceSession(self.model_path, self.device_id, self.options)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
RuntimeError: [1][ACL: invalid parameter]

Summary by CodeRabbit

  • Chores
    • Updated GitHub Actions workflows to support Ascend 910B2 hardware with expanded build configurations across multiple frameworks.
    • Refined branch trigger patterns in automated CI/CD pipelines.
    • Enhanced inline documentation in workflow configurations.

@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Nov 2, 2025
@coderabbitai

coderabbitai Bot commented Nov 2, 2025 •

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

Walkthrough

This PR updates two GitHub Actions workflows for Ascend NPU export processes. It modifies branch filter triggers (removing trailing dashes), extends export job matrices to include Ascend 910B2 processor with FunASR and framework-specific entries, and adds inline documentation comments referencing CANN image sources.

Changes

Cohort / File(s) Summary
Workflow Branch Filter and Matrix Extensions for Ascend 910B2
.github/workflows/export-paraformer-to-ascend-npu.yaml, .github/workflows/export-sense-voice-to-ascend-npu.yaml
Modified push branch filters to remove trailing dashes. Extended export-to-rknn job matrices by adding two new entries each for Ascend 910B2 with FunASR and additional framework (WSChuan-ASR / WSYue-ASR) combinations. Updated container image sections with inline comments documenting new 910B2 entries and CANN image source references.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Areas requiring attention: Verify matrix variable syntax and container image references are correctly aligned with CI/CD runner expectations; confirm branch filter changes match intended deployment branches.

Possibly related PRs

Suggested labels

size:M

Poem

🐰 Whiskers twitch at CI's delight,
910B2 shines so bright,
Workflows branch and matrices bloom,
FunASR dances through the room,
CANN tags mapped—configs zoom! ✨

✨ 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 c42a5a8 and 06e0411.

📒 Files selected for processing (2)
  • .github/workflows/export-paraformer-to-ascend-npu.yaml (3 hunks)
  • .github/workflows/export-sense-voice-to-ascend-npu.yaml (3 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 d5e5ffb into k2-fsa:master Nov 2, 2025
1 check was pending
@csukuangfj
csukuangfj deleted the export-ascend-910B2 branch November 2, 2025 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant