-
Notifications
You must be signed in to change notification settings - Fork 12
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
Performance regression and Context leak in examples/zipformer.rs since v0.5.0 #45
Comments
Hi @solaoi |
@thewh1teagle I'm using an M2 Mac, and yes, I'm using exactly the same model and example from this repository. Here are my exact steps to reproduce: # Get the code and switch to v0.5.0
git fetch
git checkout -b v0.5.0 refs/tags/v0.5.0
git submodule update --init --recursive
# Download and extract the model
wget https://github.com/k2-fsa/sherpa-onnx/releases/download/asr-models/sherpa-onnx-zipformer-ja-reazonspeech-2024-08-01.tar.bz2
tar xvf sherpa-onnx-zipformer-ja-reazonspeech-2024-08-01.tar.bz2
rm sherpa-onnx-zipformer-ja-reazonspeech-2024-08-01.tar.bz2
# Run the example
cargo run --example zipformer -- "sherpa-onnx-zipformer-ja-reazonspeech-2024-08-01/test_wavs/1.wav" "sherpa-onnx-zipformer-ja-reazonspeech-2024-08-01/encoder-epoch-99-avg-1.onnx" "sherpa-onnx-zipformer-ja-reazonspeech-2024-08-01/decoder-epoch-99-avg-1.onnx" "sherpa-onnx-zipformer-ja-reazonspeech-2024-08-01/joiner-epoch-99-avg-1.onnx" "sherpa-onnx-zipformer-ja-reazonspeech-2024-08-01/tokens.txt" |
@solaoi |
@ghjdegithub I tried running your
However, when I enabled CoreML with the command below:
I encountered the following error:
As a side note, the example command for examples/non_streaming_decode_files.rs seems to contain a typo. The documentation lists it as:
However, it should be --example |
@solaoi It does seem to be a mistake, there is a no- missing. The model you wrote in the CoreML hint is not supported, try another model, this example has a full set of parameter inputs. |
@ghjdegithub
However, I still encountered this issue error(
I also tried running it with the CPU, and that worked without any issues. Additionally, I realize I didn’t clarify my point well in the previous message. |
Description
After upgrading from v0.4.1 to v0.5.0, we've observed two significant issues in
examples/zipformer.rs
:Context leak detected, msgtracer returned -1
messages in logsPrevious Behavior (v0.4.1)
Current Behavior (v0.5.0-)
Logs
The text was updated successfully, but these errors were encountered: