Repository navigation
Upload FireRedASR v2 models (AED and CTC) - #3220
Conversation
Summary of ChangesHello @csukuangfj, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request, while titled to introduce new FireRedASR v2 models, primarily contains a minor update to a GitHub Actions test script. The change involves migrating a model download command from Highlights
Changelog
Ignored Files
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request changes a download command in a test script from wget to curl. However, there is a major discrepancy between this change and the PR's title and description, which refer to 'Upload FireRedASR v2 models'. This is misleading and should be corrected to reflect the actual change. I have also provided a suggestion to make the new curl command more robust.
| echo "----------non-streaming ASR Google MedASR CTC----------" | ||
|
|
||
| wget https://github.com/k2-fsa/sherpa-onnx/releases/download/asr-models/sherpa-onnx-medasr-ctc-en-int8-2025-12-25.tar.bz2 | ||
| curl -SL -O https://github.com/k2-fsa/sherpa-onnx/releases/download/asr-models/sherpa-onnx-medasr-ctc-en-int8-2025-12-25.tar.bz2 |
There was a problem hiding this comment.
Using -O relies on the URL's path to determine the output filename. While it works here, it's more robust to explicitly specify the output file with -o <filename>. This decouples the script from the URL structure and ensures the filename matches what subsequent commands like tar and rm expect.
| curl -SL -O https://github.com/k2-fsa/sherpa-onnx/releases/download/asr-models/sherpa-onnx-medasr-ctc-en-int8-2025-12-25.tar.bz2 | |
| curl -SL -o sherpa-onnx-medasr-ctc-en-int8-2025-12-25.tar.bz2 https://github.com/k2-fsa/sherpa-onnx/releases/download/asr-models/sherpa-onnx-medasr-ctc-en-int8-2025-12-25.tar.bz2 |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review infoConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR introduces support for FireRedASR2 ASR models through new disabled GitHub Actions packaging and publishing jobs for int8 and fp32 variants, updates publishing lists with four new model identifiers, adjusts Git LFS tracking to include "*.weights" files, and replaces a wget command with curl in the test script. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested labels
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). 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. Comment |
Closes #3219
cc @dfrrk
Download link: sherpa-onnx-fire-red-asr2-zh_en-int8-2026-02-26.tar.bz2
You can try it at

https://huggingface.co/spaces/k2-fsa/automatic-speech-recognition
Its usage is identical with
https://k2-fsa.github.io/sherpa/onnx/FireRedAsr/pretrained.html#sherpa-onnx-fire-red-asr-large-zh-en-2025-02-16-chinese-english
You don't need to change any code.
Summary by CodeRabbit