added processEOS and resolved issue #335 with unit test

[kushaljain-apra]

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).

Fixes #335

Description

Added processEOS for AudioToTextXForm module, implemented a handleflushbuffer function to clear the buffer and send an EOS frame when the buffer size is less than set buffer_size, added a unit test to check handleflushbuffer

Alternative(s) considered

Have you considered any alternatives? And if so, why have you chosen the approach in this PR?

Type

Type Choose one: Bug fix

Screenshots (if applicable)

Checklist

• I have read the Contribution Guidelines
• I have written Unit Tests
• I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach

[github-actions]

Test Results Linux

  1 files  ±0    1 suites  ±0   10m 13s ⏱️ -1s
309 tests +1  228 ✅ +1  81 💤 ±0  0 ❌ ±0 
228 runs  +1  147 ✅ +1  81 💤 ±0  0 ❌ ±0 

Results for commit 4a29bf9. ± Comparison against base commit 53157e2.

This pull request removes 1 and adds 2 tests. Note that renamed tests count towards both.

audioToTextXform_test ‑ checkEOS_asr

audioToTextXform_test ‑ check_eos_frame_asr
audioToTextXform_test ‑ check_flushed_buffer_asr

♻️ This comment has been updated with latest results.

[github-actions]

Test Results Win-nocuda

  1 files  ±0    1 suites  ±0   10m 27s ⏱️ +6s
304 tests +1  228 ✅ +1  76 💤 ±0  0 ❌ ±0 
228 runs  +1  152 ✅ +1  76 💤 ±0  0 ❌ ±0 

Results for commit 4a29bf9. ± Comparison against base commit 53157e2.

This pull request removes 1 and adds 2 tests. Note that renamed tests count towards both.

audioToTextXform_test ‑ checkEOS_asr

audioToTextXform_test ‑ check_eos_frame_asr
audioToTextXform_test ‑ check_flushed_buffer_asr

♻️ This comment has been updated with latest results.

[github-actions]

Test Results Linux-CudaT

  1 files  ±0    1 suites  ±0   10m 34s ⏱️ +5s
409 tests +1  242 ✅ +1  167 💤 ±0  0 ❌ ±0 
242 runs  +1   75 ✅ +1  167 💤 ±0  0 ❌ ±0 

Results for commit 4ab2e39. ± Comparison against base commit 5358310.

This pull request removes 1 and adds 2 tests. Note that renamed tests count towards both.

audioToTextXform_test ‑ checkEOS_asr

audioToTextXform_test ‑ check_eos_frame_asr
audioToTextXform_test ‑ check_flushed_buffer_asr

[github-actions]

Test Results Linux_ARM64

  1 files  ±0    1 suites  ±0   10m 57s ⏱️ +4s
431 tests +1  267 ✅ +1  164 💤 ±0  0 ❌ ±0 
267 runs  +1  103 ✅ +1  164 💤 ±0  0 ❌ ±0 

Results for commit 4ab2e39. ± Comparison against base commit 5358310.

This pull request removes 1 and adds 2 tests. Note that renamed tests count towards both.

audioToTextXform_test ‑ checkEOS_asr

audioToTextXform_test ‑ check_eos_frame_asr
audioToTextXform_test ‑ check_flushed_buffer_asr

[github-actions]

Test Results Windows-cuda

  1 files  ±0    1 suites  ±0   12m 20s ⏱️ +2s
405 tests +1  289 ✅ +1  116 💤 ±0  0 ❌ ±0 
289 runs  +1  173 ✅ +1  116 💤 ±0  0 ❌ ±0 

Results for commit 4a29bf9. ± Comparison against base commit 53157e2.

This pull request removes 1 and adds 2 tests. Note that renamed tests count towards both.

audioToTextXform_test ‑ checkEOS_asr

audioToTextXform_test ‑ check_eos_frame_asr
audioToTextXform_test ‑ check_flushed_buffer_asr

♻️ This comment has been updated with latest results.

[joiskash]

Looks good to me!