Increase sleep time before killing recording to resolve macOS test failures. (#917)
* Increase sleep time before killing recording to resolve macOS test failures. * Add PR #917 to changelog.ms-thread-leak-fx
parent
a2291b42dd
commit
38b20a9e56
|
@ -797,6 +797,7 @@ LDPC | Low Density Parity Check Codes - a family of powerful FEC codes
|
||||||
* Reduce usage of non-real time safe code. (PR #900)
|
* Reduce usage of non-real time safe code. (PR #900)
|
||||||
* FreeDV Reporter: Force explicit background color (avoids unwanted mouseover highlights on Linux). (PR #911)
|
* FreeDV Reporter: Force explicit background color (avoids unwanted mouseover highlights on Linux). (PR #911)
|
||||||
* Fix compiler errors when using wxWidgets 3.0. (PR #914)
|
* Fix compiler errors when using wxWidgets 3.0. (PR #914)
|
||||||
|
* Unit tests: Increase sleep time before killing recording to resolve macOS test failures. (PR #917)
|
||||||
|
|
||||||
## V2.0.0 June 2025
|
## V2.0.0 June 2025
|
||||||
|
|
||||||
|
|
|
@ -322,9 +322,11 @@ class HamlibHandler:
|
||||||
self.ErrParam()
|
self.ErrParam()
|
||||||
else:
|
else:
|
||||||
if (not x) and self.app.ptt:
|
if (not x) and self.app.ptt:
|
||||||
# Sleep for 20ms to match typical SDR behavior + 5ms to account for varying system load.
|
# Sleep for 20ms to match typical SDR behavior + 30ms to account for varying system load/virtual audio latency.
|
||||||
# Example: Flex 6000/8000 (https://community.flexradio.com/discussion/8028104/question-regarding-tx-delay)
|
# References:
|
||||||
time.sleep(25 / 1000)
|
# Virtual audio latency: https://vb-audio.com/Cable/VBCABLE_ReferenceManual.pdf (assuming 10ms/512 sample buffer size @ 48 kHz)
|
||||||
|
# Example TX->RX switching time: Flex 6000/8000 (https://community.flexradio.com/discussion/8028104/question-regarding-tx-delay)
|
||||||
|
time.sleep(50 / 1000)
|
||||||
os.kill(self.pid, signal.SIGTERM)
|
os.kill(self.pid, signal.SIGTERM)
|
||||||
if x:
|
if x:
|
||||||
self.app.ptt = 1
|
self.app.ptt = 1
|
||||||
|
|
Loading…
Reference in New Issue