pyvisa / pyvisa-py

A pure python PyVISA backend
https://pyvisa-py.readthedocs.io
MIT License
282 stars 120 forks source link

VXI11 bug fixes #322

Closed bobmcnamara closed 2 years ago

bobmcnamara commented 2 years ago
codecov-commenter commented 2 years ago

Codecov Report

Merging #322 (e0223de) into main (2dd19aa) will decrease coverage by 0.01%. The diff coverage is 8.33%.

:exclamation: Current head e0223de differs from pull request most recent head 52b8c4f. Consider uploading reports for the commit 52b8c4f to get more accurate results

@@            Coverage Diff             @@
##             main     #322      +/-   ##
==========================================
- Coverage   22.04%   22.02%   -0.02%     
==========================================
  Files          20       20              
  Lines        2595     2606      +11     
  Branches      394      397       +3     
==========================================
+ Hits          572      574       +2     
- Misses       2016     2025       +9     
  Partials        7        7              
Flag Coverage Δ
unittests 21.98% <8.33%> (-0.02%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pyvisa_py/protocols/rpc.py 22.96% <8.33%> (-0.09%) :arrow_down:

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 2dd19aa...52b8c4f. Read the comment docs.

MatthieuDartiailh commented 2 years ago

I pushed a formatting fix on main so I will merge. Thanks again for the patch !