-
Notifications
You must be signed in to change notification settings - Fork 301
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
add driver for keithley6500 #1233
add driver for keithley6500 #1233
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1233 +/- ##
=======================================
Coverage 73.22% 73.22%
=======================================
Files 79 79
Lines 9188 9188
=======================================
Hits 6728 6728
Misses 2460 2460 |
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.
Thank you! but needs some minor cleanup, see comments.
@Christian-Volk I addressed some of the comments, can you look at the remaining comments? |
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.
Now it looks better! Thanks!
Co-Authored-By: peendebak <P.T.eendebak@tudelft.nl>
Co-Authored-By: peendebak <P.T.eendebak@tudelft.nl>
Co-Authored-By: peendebak <P.T.eendebak@tudelft.nl>
Co-Authored-By: peendebak <P.T.eendebak@tudelft.nl>
Co-Authored-By: peendebak <P.T.eendebak@tudelft.nl>
Co-Authored-By: peendebak <P.T.eendebak@tudelft.nl>
fc1ab62
to
6507966
Compare
@lucblom @astafan8 @WilliamHPNielsen