Skip to content
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

[fix:cudadecoder]application hangs if numFrames == 0 #4198

Merged
merged 1 commit into from
Oct 2, 2021

Conversation

spike-liu
Copy link
Contributor

cuda_pipeline->WaitForGroup() will never return if numFrames == 0, so maintain all_group_tasks_not_done variable and notify_all if nothing is left in the group.

@spike-liu spike-liu changed the title bug fixing for cudadecoder [bug fixing:cudadecoder]application hangs if numFrames == 0 Jul 31, 2020
@spike-liu spike-liu changed the title [bug fixing:cudadecoder]application hangs if numFrames == 0 [fix:cudadecoder]application hangs if numFrames == 0 Jul 31, 2020
@hugovbraun
Copy link
Contributor

Makes sense. Thanks for your help!

@spike-liu
Copy link
Contributor Author

You are welcome, @hugovbraun.

@stale
Copy link

stale bot commented Oct 6, 2020

This issue has been automatically marked as stale by a bot solely because it has not had recent activity. Please add any comment (simply 'ping' is enough) to prevent the issue from being closed for 60 more days if you believe it should be kept open.

@stale stale bot added the stale Stale bot on the loose label Oct 6, 2020
@kkm000
Copy link
Contributor

kkm000 commented May 8, 2021

@hugovbraun, my understanding is that this code is being/has been deprecated in favor of batched-...-pipeline2.cc? Is this moot now? Is it supported on Jetson only, or not at all?

@kkm000
Copy link
Contributor

kkm000 commented Oct 2, 2021

We've deprecated batched-threaded-nnet3-cuda-pipeline.cc, but have not removed it yet. So I'm merging this fix. The CI failure is unrelated.

@stale stale bot removed the stale Stale bot on the loose label Oct 2, 2021
@kkm000 kkm000 merged commit 7547a90 into kaldi-asr:master Oct 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants