fiaisis / run-detection

The way we detect when our system for reduction should automatically perform a reduction and with what arguments we should do that.
GNU General Public License v3.0
1 stars 1 forks source link

Enable heartbeating, and reconnecting #113

Closed keiranjprice101 closed 1 year ago

keiranjprice101 commented 1 year ago

refs #26 (Fixes one side of this) closes #29

Description

Adds a graceful shutdown to queue listener. Adds heartbeating Should handle most disconnect cases (in theory)

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.57 :tada:

Comparison is base (159107c) 94.36% compared to head (b85da14) 94.93%.

:exclamation: Current head b85da14 differs from pull request most recent head 6f6ef65. Consider uploading reports for the commit 6f6ef65 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #113 +/- ## ========================================== + Coverage 94.36% 94.93% +0.57% ========================================== Files 10 10 Lines 337 375 +38 ========================================== + Hits 318 356 +38 Misses 19 19 ``` | [Impacted Files](https://app.codecov.io/gh/interactivereduction/run-detection/pull/113?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=interactivereduction) | Coverage Δ | | |---|---|---| | [rundetection/queue\_listener.py](https://app.codecov.io/gh/interactivereduction/run-detection/pull/113?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=interactivereduction#diff-cnVuZGV0ZWN0aW9uL3F1ZXVlX2xpc3RlbmVyLnB5) | `98.43% <100.00%> (+0.39%)` | :arrow_up: | | [rundetection/run\_detection.py](https://app.codecov.io/gh/interactivereduction/run-detection/pull/113?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=interactivereduction#diff-cnVuZGV0ZWN0aW9uL3J1bl9kZXRlY3Rpb24ucHk=) | `96.47% <100.00%> (+1.47%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.