New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improved error handling of B1500 driver #2041
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2041 +/- ##
==========================================
+ Coverage 71.29% 71.34% +0.05%
==========================================
Files 149 149
Lines 19945 19938 -7
==========================================
+ Hits 14220 14225 +5
+ Misses 5725 5713 -12 |
Co-authored-by: Mikhail Astafev <astafan8@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great! (waiting for last confirmation of success of last testing on the instrument )
remember to backport this to the release branch |
I created a label for that in case there are more |
The test passed. We successfully ran a measurement protocol. Also, did some tests to check if the error message is thrown with wrong settings. Good to go! |
Improved error handling of B1500 driver
Each time a parameter is set the instrument is queried for errors. If an error is found user is presented with a runtime error with an error message.