-
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
Use getattr to get ask/write from instruments #1451
Conversation
This allows us to use InstrumentBase as the base class for virtual instruments
Codecov Report
@@ Coverage Diff @@
## master #1451 +/- ##
==========================================
+ Coverage 73.24% 73.47% +0.22%
==========================================
Files 92 92
Lines 10239 10239
==========================================
+ Hits 7500 7523 +23
+ Misses 2739 2716 -23 |
Seems reasonable enough. Could you add a test based on the code in #1450 |
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.
Awesome! but indeed, please add a test for this :)
@jenshnielsen @astafan8 added some extra tests on all the changes. Not sure why the Azure test failed, I don't seem to have access to it... |
It seems to be a random networking error during the docs build. I have restarted |
|
||
|
||
class MockMetaParabola(Instrument): | ||
class MockMetaParabola(InstrumentBase): |
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.
well done finding this test suite :)
This allows us to use InstrumentBase as the base class for virtual instruments
Fixes #1450.
Changes proposed in this pull request:
@jenshnielsen @astafan8