-
Notifications
You must be signed in to change notification settings - Fork 176
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
API: adds support for pipette versions v1.3 #1474
Conversation
77f248f
to
34b5612
Compare
373748b
to
114b986
Compare
114b986
to
96429fb
Compare
changed model string to |
7c985ab
to
894ad58
Compare
…ns from QC testing
@@ -14,6 +14,7 @@ | |||
class PipetteTest(unittest.TestCase): | |||
def setUp(self): | |||
self.robot = Robot() | |||
self.robot.reset() |
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.
Not sure this does anything, but doesn't specifically need to be removed either
@@ -74,17 +76,18 @@ def test_deprecated_axis_call(self): | |||
warnings.filterwarnings('error') | |||
# Check that user warning occurs when axis is called | |||
self.assertRaises( | |||
UserWarning, Pipette, self.robot, axis='a') | |||
RuntimeError, Pipette, self.robot, mount='left') |
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.
This doesn't keep the intended effect of this test--the original purpose was to make sure that a warning would be raised if a protocol uses a deprecated parameter
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.
👍
@@ -133,8 +136,6 @@ def test_volume_percentage(self): | |||
self.assertRaises(RuntimeError, self.p200._volume_percentage, 300) | |||
self.assertEquals(self.p200._volume_percentage(100), 0.5) | |||
self.assertEquals(len(self.robot.get_warnings()), 0) | |||
self.p200._volume_percentage(self.p200.min_volume / 2) | |||
self.assertEquals(len(self.robot.get_warnings()), 1) |
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.
Are we purposefully dropping this behavior?
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.
👍
a8d6cb5
to
2d806c5
Compare
🦅 |
Merging this PR broke pipette swap in the
|
overview
This PR adds API support for pipettes marked with a version number
"v1.3"
. This new version number is needed for pipettes that are assembled with an updated homing PCB which shifts all the plunger positions by 1.5mm.Changes include:
InstrumentWrapper
, the pipette model version is derived from what pipette version is currently attached to that mount on the bot.tools.write_pipette_memory
script to write new version strings