Skip to content
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

docs(api): updated Flex default flow rates #13796

Merged
merged 1 commit into from
Oct 18, 2023
Merged

Conversation

ecormany
Copy link
Contributor

Overview

Updates the docs section on Flex Pipette Flow Rates to reflect how these values are set in production software.

Test Plan

Check out the new table in the sandbox.

Changelog

  • Replace Flex flow rates table.
  • I like en dashes (OT-2 flow rate table).
  • Clean up blow out (noun) and blowout (adj) usage.

Review requests

These figures are still good, right? Anything else we have to warn users against? Run log may or may not be reporting these figures as the default.

Risk assessment

nil, docs.

@ecormany ecormany requested a review from a team as a code owner October 16, 2023 19:51
@ecormany ecormany requested review from jwwojak and andySigler and removed request for a team October 16, 2023 19:52
@codecov
Copy link

codecov bot commented Oct 16, 2023

Codecov Report

Merging #13796 (576308d) into edge (60aaf7d) will not change coverage.
Report is 1 commits behind head on edge.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             edge   #13796   +/-   ##
=======================================
  Coverage   70.75%   70.75%           
=======================================
  Files        2465     2465           
  Lines       69346    69346           
  Branches     8338     8338           
=======================================
  Hits        49069    49069           
  Misses      18317    18317           
  Partials     1960     1960           
Flag Coverage Δ
g-code-testing 96.44% <ø> (ø)
notify-server 89.13% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@@ -345,11 +345,11 @@ If you know that all your liquid handling will take place in a specific mode, th
Pipette Flow Rates
==================

Measured in µL/s, the flow rate determines how much liquid a pipette can aspirate, dispense, and blow-out. Opentrons pipettes have their own default flow rates. The API lets you change the flow rate on a loaded :py:class:`.InstrumentContext` by altering the :py:obj:`.InstrumentContext.flow_rate` properties listed below.
Measured in µL/s, the flow rate determines how much liquid a pipette can aspirate, dispense, and blow out. Opentrons pipettes have their own default flow rates. The API lets you change the flow rate on a loaded :py:class:`.InstrumentContext` by altering the :py:obj:`.InstrumentContext.flow_rate` properties listed below.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"blow out" oops, my bad. 🥇

Copy link
Contributor

@jwwojak jwwojak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice updates to have.

:octocat:

@ecormany ecormany added docs papi-v2 Python API V2 labels Oct 17, 2023
@ecormany ecormany merged commit e960aaa into edge Oct 18, 2023
39 checks passed
@ecormany ecormany deleted the RTC-329-updated-flow-rates branch October 18, 2023 13:53
y3rsh added a commit that referenced this pull request Oct 25, 2023
* edge: (77 commits)
  fix(app): update move gantry text in change pipette flow (#13712)
  fix(app): fix CI after release merge back (#13816)
  chore(hardware): add opentrons hardware package to ot 2 build (#13770)
  feat(protocol-designer): correct step count in create file wizard (#13807)
  feat(protocol-designer): error handling in create file wizard (#13804)
  fix(app): invalidate OT2 calibration queries when calibration flows complete (#13809)
  always jog (#13806)
  chore(app): point updates at dns not s3 (#13798)
  docs(api): updated Flex default flow rates (#13796)
  fix(api): Flag pipette as not ready to aspirate after pushing out air (#13728)
  fix(api, hardware): allow OT3Controller to disable tip motors (#13805)
  more blank trials; longer scale stabilize wait; submerge 2.5mm (#13788)
  fix(app): unload adapters after checking position of labware on adapter on heater shaker module (#13803)
  feat(app): wire up location conflict modal for ODD (#13797)
  feat(app): add Deck configuration page component (#13784)
  fix(robot-server): add default to robot serial (#13802)
  feat(protocol-designer, components): prep work for deck config in PD (#13775)
  feat(app, protocol-designer): plug in waste chute asset (#13800)
  fix(app): do not check mag block in calibration status (#13799)
  feat(protocol-designer): edit additional items staging area support (#13752)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs papi-v2 Python API V2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants