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

chore: known issues 7.2.0 – March 4 #14596

Merged
merged 1 commit into from
Mar 4, 2024

Conversation

ecormany
Copy link
Contributor

@ecormany ecormany commented Mar 4, 2024

Overview

Added a couple known issues that won't be addressed in 7.2.0.

Review requests

Both issues sufficiently and accurately summarized?

@ecormany ecormany requested review from sfoster1 and nusrat813 March 4, 2024 22:36
@ecormany ecormany requested a review from a team as a code owner March 4, 2024 22:36
@ecormany ecormany requested review from jerader and removed request for a team March 4, 2024 22:36
### Known Issues

- It's possible to start conflicting instrument detachment workflows when controlling one robot from multiple computers. Verify that the robot is idle before starting instrument detachment.
- Robots may fail to reconnect after renaming them over a USB connection on Windows.
Copy link
Member

Choose a reason for hiding this comment

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

This doesn't seem to actually be an issue anymore right @skowalski08

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I based these on @nusrat813's categorization of RQA-2300 and RQA-2426. Happy to change if either is fixed!

Copy link
Contributor

@skowalski08 skowalski08 Mar 4, 2024

Choose a reason for hiding this comment

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

This ticket came up recently:

https://opentrons.atlassian.net/browse/RQA-2300

Rob should be in the office tomorrow, so he can retest. @sfoster1

Copy link
Member

Choose a reason for hiding this comment

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

Ok, LGTM then

Copy link

codecov bot commented Mar 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.57%. Comparing base (45e302a) to head (2bbd622).

Additional details and impacted files

Impacted file tree graph

@@                 Coverage Diff                  @@
##           chore_release-7.2.0   #14596   +/-   ##
====================================================
  Coverage                67.57%   67.57%           
====================================================
  Files                     2521     2521           
  Lines                    72241    72241           
  Branches                  9306     9306           
====================================================
  Hits                     48820    48820           
  Misses                   21226    21226           
  Partials                  2195     2195           
Flag Coverage Δ
app 63.95% <ø> (ø)

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

Copy link
Member

@sfoster1 sfoster1 left a comment

Choose a reason for hiding this comment

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

LGTM!

@sfoster1 sfoster1 merged commit be34672 into chore_release-7.2.0 Mar 4, 2024
28 checks passed
@sfoster1 sfoster1 deleted the known-issues-7.2.0-march-4 branch March 4, 2024 23:25
@ecormany
Copy link
Contributor Author

ecormany commented Mar 5, 2024

Thanks for getting this into alpha 12.

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

3 participants