fix(hardware-testing): updates to the hardware-testing package for the new QC protocol (#18601)

<!--
Thanks for taking the time to open a Pull Request (PR)! Please make sure
you've read the "Opening Pull Requests" section of our Contributing
Guide:


https://github.com/Opentrons/opentrons/blob/edge/CONTRIBUTING.md#opening-pull-requests

GitHub provides robust markdown to format your PR. Links, diagrams,
pictures, and videos along with text formatting make it possible to
create a rich and informative PR. For more information on GitHub
markdown, see:


https://docs.github.com/en/get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax

To ensure your code is reviewed quickly and thoroughly, please fill out
the sections below to the best of your ability!
-->

# Overview

We needed a few more updates to the hardware-testing repository to make
the gravimetric fixture work. Due to the way that it pulls in the
hardware-testing package that is tagged with the release, even though we
don't distribute hardware-testing with the normal distribution we need
keep the updates in sync with the release tag.

This PR is mainly focused at getting the changes to ui.py in so that we
can have access to the output from hardware-testing package for a
particular run put together with the results of that script run.

This also adds an overloaded setter to TipPosition in
_liquid_properites.py so that we can maintain type consistency for
linting, and continue to have an argument option for customers so they
don't need to import a specific type.
<!--
Describe your PR at a high level. State acceptance criteria and how this
PR fits into other work. Link issues, PRs, and other relevant resources.
-->

## Test Plan and Hands on Testing

<!--
Describe your testing of the PR. Emphasize testing not reflected in the
code. Attach protocols, logs, screenshots and any other assets that
support your testing.
-->

## Changelog

<!--
List changes introduced by this PR considering future developers and the
end user. Give careful thought and clear documentation to breaking
changes.
-->

## Review requests

<!--
- What do you need from reviewers to feel confident this PR is ready to
merge?
- Ask questions.
-->

## Risk assessment

<!--
- Indicate the level of attention this PR needs.
- Provide context to guide reviewers.
- Discuss trade-offs, coupling, and side effects.
- Look for the possibility, even if you think it's small, that your
change may affect some other part of the system.
- For instance, changing return tip behavior may also change the
behavior of labware calibration.
- How do your unit tests and on hands on testing mitigate this PR's
risks and the risk of future regressions?
- Especially in high risk PRs, explain how you know your testing is
enough.
-->

Author: ryanthecoder

api/src/opentrons/protocol_api/_liquid_properties.py

@@ -115,16 +115,28 @@ def position_reference(self) -> PositionReference:
         return self._position_reference
 
     @position_reference.setter
-    def position_reference(self, new_position: str) -> None:
-        self._position_reference = PositionReference(new_position)
+    def position_reference(self, new_position: Union[str, PositionReference]) -> None:
+        self._position_reference = (
+            new_position
+            if isinstance(new_position, PositionReference)
+            else PositionReference(new_position)
+        )
 
     @property
     def offset(self) -> Coordinate:
         return self._offset
 
     @offset.setter
-    def offset(self, new_offset: Sequence[float]) -> None:
-        x, y, z = validation.validate_coordinates(new_offset)
+    def offset(self, new_offset: Union[Sequence[float], Coordinate]) -> None:
+        if isinstance(new_offset, Coordinate):
+            new_coordinate: Sequence[Union[int, float]] = [
+                new_offset.x,
+                new_offset.y,
+                new_offset.z,
+            ]
+        else:
+            new_coordinate = new_offset
+        x, y, z = validation.validate_coordinates(new_coordinate)
         self._offset = Coordinate(x=x, y=y, z=z)
 
     def as_shared_data_model(self) -> SharedDataTipPosition:

hardware-testing/Makefile

@@ -216,7 +216,7 @@ push-ot3:
 	$(MAKE) push-plot-webpage-ot3
 	$(MAKE) push-description-ot3
 	$(MAKE) push-labware-ot3
-	cd ../ && $(MAKE) -C shared-data push-ot3
+	#cd ../ && $(MAKE) -C shared-data push-ot3
 
 .PHONE: open-dev-app
 open-dev-app:

hardware-testing/hardware_testing/data/ui.py

@@ -1,13 +1,36 @@
 """Production QC User Interface."""
 from opentrons.hardware_control import SyncHardwareAPI
 from opentrons.hardware_control.types import StatusBarState
+from typing import Optional
 
 PRINT_HEADER_NUM_SPACES = 4
 PRINT_HEADER_DASHES = "-" * PRINT_HEADER_NUM_SPACES
 PRINT_TITLE_POUNDS = "#" * PRINT_HEADER_NUM_SPACES
 PRINT_HEADER_SPACES = " " * (PRINT_HEADER_NUM_SPACES - 1)
 PRINT_HEADER_ASTERISK = "*"
 
+outfile: Optional[str] = None
+
+
+def set_output_file(new_outfile: Optional[str]) -> None:
+    """Change the output location of the UI output.
+
+    If it is a string it will output to that as a file.
+    if it is None it will default back to stdout
+    """
+    global outfile
+    outfile = new_outfile
+    _output(f"Setting UI output to file {outfile}")
+
+
+def _output(msg: str) -> None:
+    global outfile
+    if outfile:
+        with open(outfile, "a") as f:
+            f.write(f"{msg}\n")
+    else:
+        print(msg)
+
 
 def get_user_answer(question: str) -> bool:
     """Get user answer."""
@@ -23,7 +46,9 @@ def get_user_answer(question: str) -> bool:
 
 def get_user_ready(message: str) -> None:
     """Get user ready."""
-    input(f"WAIT: {message}, press ENTER when ready: ")
+    global outfile
+    if not outfile:
+        input(f"WAIT: {message}, press ENTER when ready: ")
 
 
 def alert_user_ready(message: str, hw: SyncHardwareAPI) -> None:
@@ -43,7 +68,7 @@ def print_title(title: str) -> None:
     length = len(title)
     pounds = PRINT_TITLE_POUNDS + ("#" * length) + PRINT_TITLE_POUNDS
     middle = f"#{PRINT_HEADER_SPACES}" f"{title}" f"{PRINT_HEADER_SPACES}#"
-    print(f"\n{pounds}\n{middle}\n{pounds}\n")
+    _output(f"\n{pounds}\n{middle}\n{pounds}\n")
 
 
 def print_header(header: str) -> None:
@@ -56,19 +81,19 @@ def print_header(header: str) -> None:
     length = len(header)
     dashes = PRINT_HEADER_DASHES + ("-" * length) + PRINT_HEADER_DASHES
     middle = f"|{PRINT_HEADER_SPACES}{header}{PRINT_HEADER_SPACES}|"
-    print(f"\n{dashes}\n{middle}\n{dashes}\n")
+    _output(f"\n{dashes}\n{middle}\n{dashes}\n")
 
 
 def print_error(message: str) -> None:
     """Print error."""
-    print(f"ERROR: {message}")
+    _output(f"ERROR: {message}")
 
 
 def print_warning(message: str) -> None:
     """Print warning."""
-    print(f"WARNING: {message}")
+    _output(f"WARNING: {message}")
 
 
 def print_info(message: str) -> None:
     """Print information."""
-    print(message)
+    _output(message)

hardware-testing/hardware_testing/gravimetric/protocol_replacement/96ch1000.csv

@@ -12,7 +12,7 @@ tipracks_50ul,D2
 tipracks_200ul,,,,,,,,,
 tipracks_1000ul,,,,,,,,,
 labware_on_scale,nest_1_reservoir_195ml,A1,,,,,,,
-slot_scale,B1,,,,,,,,
+slot_scale,C1,,,,,,,,
 volumes_to_test_20ul,,,,,,,,,
 volumes_to_test_50ul,50,,,,,,,,
 volumes_to_test_200ul,,,,,,,,,

hardware-testing/hardware_testing/gravimetric/protocol_replacement/96ch200.csv

@@ -12,7 +12,7 @@ tipracks_50ul,D2,D3,C2,C3,B1,B2,B3,A1,A2
 tipracks_200ul,,,,,,,,,
 tipracks_1000ul,,,,,,,,,
 labware_on_scale,nest_1_reservoir_195ml,A1,,,,,,,
-slot_scale,B1,,,,,,,,
+slot_scale,C1,,,,,,,,
 volumes_to_test_20ul,,,,,,,,,
 volumes_to_test_50ul,50,,,,,,,,
 volumes_to_test_200ul,,,,,,,,,