-
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
updated/fixed ATS9870 driver #1290
Conversation
This replaces #1183 |
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.
Looks correct to me
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.
@wpfff Thanks a lot, especially for testing! We'll do the same for the other model, and the remove AlazarParameter completely (see other @jenshnielsen PRs)
Codecov Report
@@ Coverage Diff @@
## master #1290 +/- ##
==========================================
- Coverage 70.92% 70.91% -0.02%
==========================================
Files 74 74
Lines 8352 8352
==========================================
- Hits 5924 5923 -1
- Misses 2428 2429 +1 |
@wpfff The last commit does not look like it belongs on this branch |
@jenshnielsen we reverted those commits. their content is indeed in a separate PR. |
Great, only remaining issue is a trailing whitespace on line 28 but I am willing to let that slip and fix it in #1184 |
updated the ATS9870 to new conventions; was broken with recent changes to the ATS base driver.
this uses TraceParameter now. tested on LK1 in Delft.
@jenshnielsen @astafan8