-
Notifications
You must be signed in to change notification settings - Fork 175
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
fix(app): configure modules during calibration, shorten heater shaker fixture name #14953
Conversation
… fixture name At the beginning of the module calibration flow, the user is asked to locate the module on the deck. This integrated the deck configurator component directly into this location selction step of the module calibration wizard. the selected location will now be saved directly to deck configuration. Closes RQA-2603
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## edge #14953 +/- ##
==========================================
+ Coverage 67.50% 76.11% +8.60%
==========================================
Files 2521 41 -2480
Lines 72090 2746 -69344
Branches 9311 0 -9311
==========================================
- Hits 48666 2090 -46576
+ Misses 21228 656 -20572
+ Partials 2196 0 -2196
Flags with carried forward coverage won't be shown. Click here to find out more. |
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.
Overall looks solid, app flow makes sense when using and I was able to successfully calibrate a thermocycler on a Flex with no interruptions.
const cutoutId = deckConfig.find( | ||
cc => cc.opentronsModuleSerialNumber === attachedModule.serialNumber | ||
cc => | ||
cc.opentronsModuleSerialNumber === attachedModule.serialNumber && | ||
(attachedModule.moduleType !== THERMOCYCLER_MODULE_TYPE || | ||
cc.cutoutFixtureId === THERMOCYCLER_V2_FRONT_FIXTURE) |
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 fix looks appropriate and solves the issues we were having in the case of multiple thermocycler fixtures proccing on the front end from the deck config.
// If module fixture is loaded, still visualize singleSlotFixture underneath for consistency | ||
Object.entries(MODULE_FIXTURES_BY_MODEL).reduce<CutoutFixtureId[]>( | ||
(acc, [_model, fixtures]) => [...acc, ...fixtures], | ||
[] | ||
)) |
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.
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.
interesting..., yeah maybe it should only be done for module fixtures. Good catch! I'll patch it up
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.
LGTM !
… fixture name (#14953) # Overview At the beginning of the module calibration flow, the user is asked to locate the module on the deck. This integrated the deck configurator component directly into this location selction step of the module calibration wizard. the selected location will now be saved directly to deck configuration. Closes [RQA-2603](https://opentrons.atlassian.net/browse/RQA-2603) # Review requests - Run module calibration and confirm that the selected location reflects the deck configuration # Risk assessment low [RQA-2603]: https://opentrons.atlassian.net/browse/RQA-2603?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
Overview
At the beginning of the module calibration flow, the user is asked to locate the module on the deck.
This integrated the deck configurator component directly into this location selction step of the
module calibration wizard. the selected location will now be saved directly to deck configuration.
Closes RQA-2603
Review requests
Risk assessment
low