pymeasure / pyleco

Python implementation of the Laboratory Experiment COntrol (LECO) protocol
MIT License
9 stars 3 forks source link

Rework communicator #48

Closed BenediktBurger closed 8 months ago

BenediktBurger commented 8 months ago

Improve the inner workings of the communicator and message handler (using the same base code).

codecov[bot] commented 8 months ago

Codecov Report

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

Comparison is base (54f7d2b) 86.47% compared to head (a777f6a) 87.08%.

Files Patch % Lines
pyleco/utils/communicator.py 96.42% 1 Missing :warning:
pyleco/utils/extended_message_handler.py 0.00% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #48 +/- ## ========================================== + Coverage 86.47% 87.08% +0.60% ========================================== Files 32 33 +1 Lines 2581 2531 -50 Branches 312 300 -12 ========================================== - Hits 2232 2204 -28 + Misses 290 272 -18 + Partials 59 55 -4 ``` | [Flag](https://app.codecov.io/gh/pymeasure/pyleco/pull/48/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pymeasure) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/pymeasure/pyleco/pull/48/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pymeasure) | `87.08% <98.60%> (+0.60%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pymeasure#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.