search

jitsi / jigasi

Jigasi: a server-side application acting as a gateway to Jitsi Meet conferences. Currently allows regular SIP clients to join meetings and provides transcription capabilities.
Apache License 2.0
525 stars 295 forks source link

fix: wrong transcription order #520

Closed ltorje-8x8 closed 7 months ago

ltorje-8x8 commented 7 months ago

The receiving timestamp will be sent to skynet to be used as absolute timestamp for the audio chunk - the network lag is presumed to be less than tens of milliseconds (a latency of 20-50 ms is expected and it is considered to be ok as loss of time).

codecov[bot] commented 7 months ago

Codecov Report

Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (86273ae) 20.86% compared to head (37b3e33) 21.85%. Report is 6 commits behind head on master.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/jitsi/jigasi/pull/520/graphs/tree.svg?width=650&height=150&src=pr&token=bRTGRUW8Sz&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi)](https://app.codecov.io/gh/jitsi/jigasi/pull/520?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi) ```diff @@ Coverage Diff @@ ## master #520 +/- ## ============================================ + Coverage 20.86% 21.85% +0.99% - Complexity 289 309 +20 ============================================ Files 74 76 +2 Lines 6222 6269 +47 Branches 833 838 +5 ============================================ + Hits 1298 1370 +72 + Misses 4699 4671 -28 - Partials 225 228 +3 ``` | [Files](https://app.codecov.io/gh/jitsi/jigasi/pull/520?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi) | Coverage Δ | | |---|---|---| | [...va/org/jitsi/jigasi/transcription/Transcriber.java](https://app.codecov.io/gh/jitsi/jigasi/pull/520?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi#diff-c3JjL21haW4vamF2YS9vcmcvaml0c2kvamlnYXNpL3RyYW5zY3JpcHRpb24vVHJhbnNjcmliZXIuamF2YQ==) | `0.00% <ø> (ø)` | | | [...g/jitsi/jigasi/transcription/WhisperWebsocket.java](https://app.codecov.io/gh/jitsi/jigasi/pull/520?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi#diff-c3JjL21haW4vamF2YS9vcmcvaml0c2kvamlnYXNpL3RyYW5zY3JpcHRpb24vV2hpc3BlcldlYnNvY2tldC5qYXZh) | `28.39% <46.15%> (+28.39%)` | :arrow_up: | ... and [8 files with indirect coverage changes](https://app.codecov.io/gh/jitsi/jigasi/pull/520/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi) ------ [Continue to review full report in Codecov by Sentry](https://app.codecov.io/gh/jitsi/jigasi/pull/520?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://app.codecov.io/gh/jitsi/jigasi/pull/520?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi). Last update [0a047ff...37b3e33](https://app.codecov.io/gh/jitsi/jigasi/pull/520?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi).