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

Protocol designer OT-3 development #12540

Closed

Conversation

akshay-dighe
Copy link

@akshay-dighe akshay-dighe commented Apr 21, 2023

Overview

• Landing page added with three buttons.
• Implement pipette and tip rack selection based on the Figma design.
• Implement nav pill form according to the Figma design.
• Create new routes based on the Figma design.
• Note: this is ongoing development

Implementation details
• Implement routing for the new routes using React Router.
• OT-2 Button navigates previous OT-2 platform.
• Create appropriate route components and configure routing paths.
• Added new components for form data collection.
• Added new components for pipette and tip rack selection.
• For the nav pill form state management Formik state management and props
• Added new components for the nav pill form.
• Showing dynamic module selection (not added gripper and trash chute)
• Dynamic Pipette selection and design changes.
• Pipette validation
• Import OT2 and OT3 files
• file details page for the new OT3 json.
• Navigate to liquid page functionality on module selection.

Test Plan

Working on JSON creations, right now unable to test.

Changelog

• Implement routing for the new routes using React Router.
• OT-2 Button navigates previous OT-2 platform.
• Create appropriate route components and configure routing paths.
• Added new components for form data collection.
• Added new components for pipette and tip rack selection.
• For the nav pill form state management Formik state management and props
• Added new components for the nav pill form.
• Showing dynamic module selection (not added gripper and trash chute)
• Dynamic Pipette selection and design changes.
• Pipette validation
• Import OT2 and OT3 files
• file details page for the new OT3 json.
• Navigate to liquid page functionality on module selection.

Review requests

Risk assessment

Medium: module selection development is ongoing and OT3 JSON creation not completed yet.
• OT3 file edit functionality is InProgress.
• Ot2 and OT3 supported tip rack differentiation InProgress.

aditya-sancheti and others added 30 commits April 7, 2023 10:59
* Landing Page Initial Commit

* Round Tabs implementation
Revert pipette selection component as previous commit
Copy link
Collaborator

@jerader jerader left a comment

Choose a reason for hiding this comment

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

looking good, thanks for addressing a lot of my comments. I think the next big step is being able to successfully export a protocol that creates a json file with the Opentrons Flex robot for the robot type and the Flex pipettes.

components/src/molecules/RoundTab.stories.tsx Outdated Show resolved Hide resolved
protocol-designer/src/labware-defs/utils.ts Outdated Show resolved Hide resolved
protocol-designer/src/containers/ConnectedTitleBar.tsx Outdated Show resolved Hide resolved
protocol-designer/src/components/ProtocolEditor.css Outdated Show resolved Hide resolved
Copy link
Collaborator

@jerader jerader left a comment

Choose a reason for hiding this comment

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

I left some questions about why things are commented out. If you are going to use something in a follow up PR that is currently commented out, please leave a short comment describing what the purpose of it is, that makes it easier to review.

Additionally, the changes made to the labware-defs/utils affects OT2 PD. We should keep OT2 PD functionality the same. Probably the best solution is to modify the util to work for both OT2 and OT3

protocol-designer/src/labware-defs/utils.ts Show resolved Hide resolved
protocol-designer/src/configureStore.ts Outdated Show resolved Hide resolved
Comment on lines +152 to +157
// const [customTipRackPropsUpdated, setCustomTipRackProps] = useState(
// customTipRackProps
// )
// useEffect(() => {
// setCustomTipRackProps(customTipRackProps)
// }, [customTipRackProps])
Copy link
Collaborator

Choose a reason for hiding this comment

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

can this be deleted?

Copy link
Author

Choose a reason for hiding this comment

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

Yes. In the next pull request, all irrelevant code or non-essential elements will be removed.

import * as React from 'react'
import { ProtocolEditor } from './ProtocolEditor'

import React from 'react'
Copy link
Contributor

Choose a reason for hiding this comment

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

is there a reason why the react import statement needed to be changed here?

Copy link
Author

Choose a reason for hiding this comment

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

I will revert the change in this component since it is not necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants