Closed matthias-schuessler closed 1 year ago
Base: 35.90% // Head: 34.44% // Decreases project coverage by -1.46%
:warning:
Coverage data is based on head (
fc6d226
) compared to base (26fcc9f
). Patch coverage: 0.00% of modified lines in pull request are covered.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Thanks for the contribution, looks good! Would be nice if you could add an example to the examples folder. If you don't find any of the existing folders fitting (e.g. lab_devices
, etc) you can also add a new folder (e.g. EOS
or serial_powering
, etc). The example should consist of a config-yaml as well as a Python script showcasing the usage and basic functionality of the contents of the XPT.py
HL.
I talked with Florian and Yannick about adding an example. We think that it doesn't make sense to add one, since our only application for the switches is the QMS-card, which needs bdaq to operate. Thus, an example wouldn't work in basil without adding the entire bdaq class.
Okay, then let me know once this is ready to be merged
From my end, this is ready to merge.
Adds a file XPT.py containing the HL for EOS and QMS card.
Also contains a change to Keithley2000 adding the option to set NPLC in DC voltage mode