Closed wtgee closed 3 years ago
Merging #274 (4983b4e) into develop (3791a1d) will increase coverage by
1.07%
. The diff coverage is86.82%
.
@@ Coverage Diff @@
## develop #274 +/- ##
===========================================
+ Coverage 73.56% 74.64% +1.07%
===========================================
Files 30 31 +1
Lines 2001 2086 +85
===========================================
+ Hits 1472 1557 +85
Misses 529 529
Impacted Files | Coverage Δ | |
---|---|---|
...utils/serial/handlers/protocol_arduinosimulator.py | 19.12% <30.76%> (ø) |
|
...c/panoptes/utils/serial/handlers/protocol_no_op.py | 75.00% <33.33%> (ø) |
|
...panoptes/utils/serial/handlers/protocol_buffers.py | 81.81% <78.57%> (ø) |
|
src/panoptes/utils/rs232.py | 88.79% <100.00%> (+0.40%) |
:arrow_up: |
src/panoptes/utils/serial/device.py | 100.00% <100.00%> (ø) |
|
src/panoptes/utils/serial/handlers/__init__.py | 100.00% <100.00%> (ø) |
|
.../panoptes/utils/serial/handlers/protocol_hooked.py | 100.00% <100.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 3791a1d...4983b4e. Read the comment docs.
The
SerialDevice
class will eventually replace thers232.SerialData
class, which has a number of baked-in problems. Specifically, the newSerialDevice
features a threaded reader class that allows for custom parsing. Default processing puts the serial entries in areadings
deque.serial_handlers
intopanoptes.utils.serial
namespace.rs232
module as deprecated.See example notebook and docstrings for usage.