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

feature(hardware): add a warning style to can_mon and an "estop_released" error id #11924

Merged
merged 3 commits into from
Jan 3, 2023

Conversation

ryanthecoder
Copy link
Contributor

Overview

Changelog

Review requests

Risk assessment

@ryanthecoder ryanthecoder requested a review from a team as a code owner December 22, 2022 16:02
@codecov
Copy link

codecov bot commented Dec 22, 2022

Codecov Report

Merging #11924 (4a54336) into edge (82d91cb) will increase coverage by 0.01%.
The diff coverage is 14.28%.

Impacted file tree graph

@@            Coverage Diff             @@
##             edge   #11924      +/-   ##
==========================================
+ Coverage   74.23%   74.24%   +0.01%     
==========================================
  Files        1431     2167     +736     
  Lines       47350    60006   +12656     
  Branches     3013     6307    +3294     
==========================================
+ Hits        35148    44554    +9406     
- Misses      11735    13964    +2229     
- Partials      467     1488    +1021     
Flag Coverage Δ
app 73.18% <ø> (+12.10%) ⬆️

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

Impacted Files Coverage Δ
.../opentrons_hardware/firmware_bindings/constants.py 100.00% <ø> (ø)
hardware/opentrons_hardware/scripts/can_mon.py 0.00% <0.00%> (ø)
...rdware/opentrons_hardware/drivers/gpio/__init__.py 47.05% <100.00%> (ø)
...rganisms/Devices/hooks/useRunCreatedAtTimestamp.ts 66.66% <0.00%> (ø)
.../src/organisms/CalibrationPanels/MeasureNozzle.tsx 71.42% <0.00%> (ø)
...rganisms/Devices/hooks/useRunPipetteInfoByMount.ts 65.30% <0.00%> (ø)
...pp/src/organisms/ModuleCard/MagneticModuleData.tsx 80.00% <0.00%> (ø)
... and 732 more

Copy link
Contributor

@Laura-Danielle Laura-Danielle left a comment

Choose a reason for hiding this comment

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

lgtm

@ryanthecoder ryanthecoder force-pushed the RET-1252-estop-released-id branch from 83a05a1 to 4a54336 Compare January 3, 2023 17:05
@ryanthecoder ryanthecoder merged commit 9401c52 into edge Jan 3, 2023
y3rsh added a commit that referenced this pull request Jan 6, 2023
* origin/edge: (34 commits)
  refactor(app): update desktop robot settings calibration section for OT-3 (#11942)
  feat(hardware): add CAN message to update motor position from encoders (#11868)
  ci(monorepo): upgrade windows versions on github workflows (#11940)
  feat(app): implement useCalibrationTaskList hook (#11894)
  feat(app, app-shell, app-shell-odd): create node layer for ODD (#11944)
  refactor(app): Remove recalibrate option from POC and TLC overflow menus [RAUT-93] (#11915)
  fix(api): home z should home gripper z too (#11950)
  refactor(shared-data): gripper use force polynomial function  (#11946)
  refactor(api): move `ModuleGeometry` to legacy protocol core module (#11939)
  refactor(api): deprecate `ModuleContext.geometry` (#11938)
  chore(usb-bridge): add usb-bridge tests to test-py, add restart to push-ot3 (#11937)
  Revert "feat(app-shell-odd): create node layer for ODD (#11852)" (#11941)
  feat(app-shell-odd): create node layer for ODD (#11852)
  feature(hardware): add a warning style to can_mon and an "estop_released" error id (#11924)
  fix(hardware): Remove while loop and rely on  number_of_messages when parsing motor position response.  (#11929)
  fix(hardware): save can_comm / can_mon logs to read-write location (#11933)
  feat(api): Support 96 channel in the hardware controller (#11866)
  refactor(app): revert run a protocol from devices pages (#11909)
  refactor(app): remove warnings (#11922)
  refactor(app): remove invalid type warnings for strings from atoms (#11918)
  ...
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.

2 participants