-
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
Dev/b2200 switch matrix multiple channel connect #1480
Dev/b2200 switch matrix multiple channel connect #1480
Conversation
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 great! will merge once CI is green))
@@ -4,6 +4,7 @@ | |||
|
|||
from qcodes import VisaInstrument | |||
from qcodes.utils.validators import MultiType, Ints, Enum, Lists | |||
from typing import List, Tuple | |||
|
|||
|
|||
def post_execution_status_poll(func): |
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.
as the mypy output of the CI suggests, you need to add type hints to this decorator. you can read about this online, but perhaps smth simple like def post_execution_status_poll(func: Callable) -> Callable:
could work.
Codecov Report
@@ Coverage Diff @@
## master #1480 +/- ##
==========================================
+ Coverage 73.88% 73.91% +0.03%
==========================================
Files 92 92
Lines 10420 10433 +13
==========================================
+ Hits 7699 7712 +13
Misses 2721 2721 |
New feature for B2200 switch matrix driver:
Changes proposed in this pull request:
connect_paths()
anddisconnect_paths()
)connect()
multiple times in a loop.Includes tests and updated example notebook.
@astafan8