Repository navigation
cuda: flush temporal extractors once with a thread pool - #1553
BardieJoensen wants to merge 1 commit into
Conversation
Skip CUDA extractors in the temporal flush loop to avoid duplicate final scores. Add a regression across thread counts and short clips.
64c7845 to
4b2fe89
Compare
|
Tested on master The regression test fails without the fix and passes with it. Through the CLI: on master, Overlap: the same guard is in the first commit of #1583 ( |
With CPU threads enabled,
motion_cudais flushed by both the temporal loop and the CUDA loop. The second flush appends the final score again and fails with-EINVAL.This change skips CUDA extractors in the temporal loop and adds a regression test covering 0/1/4 threads, 1/2/12 frames, 8/10-bit input and mixed CPU/CUDA extraction. The test checks every final score. It failed before the fix and passed afterward.
Related: #1538. This fix preserves CPU threading.