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

feat(hardware): add acceleration to pick up tip for 96 channel #12944

Merged
merged 28 commits into from
Jul 27, 2023
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
95d068b
add acceleration to tip_action
caila-marashaj Jun 15, 2023
b4d4bee
pick up tip move happens, but rumbly
caila-marashaj Jun 21, 2023
bdea049
linter stuff cleanup
caila-marashaj Jun 21, 2023
782a760
fixed position of acceleration in tipaction msg
caila-marashaj Jun 22, 2023
73d26bf
moving but delayed
caila-marashaj Jun 27, 2023
c3e044a
modify tip action, debugging code still in there
caila-marashaj Jun 28, 2023
a85074e
mostly working but returning position too early
caila-marashaj Jun 29, 2023
1952348
need to update tests
caila-marashaj Jul 6, 2023
fa194a3
updated unit test args for tip_action
caila-marashaj Jul 17, 2023
d2e7927
rebase
caila-marashaj Jul 17, 2023
2b20a20
rough final draft
caila-marashaj Jul 18, 2023
8cf3c15
cleanup
caila-marashaj Jul 18, 2023
7e5b7ef
format
caila-marashaj Jul 18, 2023
8ace0ef
remove redundant data check from gear motors
caila-marashaj Jul 18, 2023
155738b
hardware controller changes
caila-marashaj Jul 19, 2023
0aede03
increase backward distances
caila-marashaj Jul 20, 2023
97e12e9
changed helpers_ot3 function
caila-marashaj Jul 21, 2023
1550c9a
format
caila-marashaj Jul 21, 2023
735ebe7
helpers_ot3 function change
caila-marashaj Jul 21, 2023
af1eefa
got rid of copysign call
caila-marashaj Jul 21, 2023
e2280aa
check if speed exists
caila-marashaj Jul 21, 2023
5be4492
add deafult value to axis convert call
caila-marashaj Jul 24, 2023
93ef4ca
update gear motor position handling after move
caila-marashaj Jul 25, 2023
f1a42d1
test fix
caila-marashaj Jul 25, 2023
2b71f07
moved comments
caila-marashaj Jul 26, 2023
57bb955
remove updategearmotorpositionestimation msg
caila-marashaj Jul 26, 2023
e201a43
removed more unneeded code
caila-marashaj Jul 26, 2023
6eb76e6
address change reqs
caila-marashaj Jul 27, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fixed position of acceleration in tipaction msg
  • Loading branch information
caila-marashaj committed Jul 26, 2023
commit 782a7603d74dca4fb3301d76b0ec473ea4ca63ed
2 changes: 1 addition & 1 deletion api/src/opentrons/hardware_control/ot3api.py
Original file line number Diff line number Diff line change
Expand Up @@ -1681,7 +1681,7 @@ async def _motor_pick_up_tip(

gear_motor_origin = await self._backend.gear_motor_position_estimation()
gear_origin_dict = {OT3Axis.Q: gear_motor_origin[0]}
gear_motor_target = pipette_spec.pick_up_distance + pipette_spec.home_buffer
gear_motor_target = pipette_spec.pick_up_distance
gear_target_dict = {OT3Axis.Q: gear_motor_target}
moves = self._build_moves(gear_origin_dict, gear_target_dict)
blocks = moves[0][0].blocks
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -555,9 +555,9 @@ class TipActionRequestPayload(AddToMoveGroupRequestPayload):
"""A request to perform a tip action."""

velocity: utils.Int32Field
acceleration: utils.Int32Field
action: PipetteTipActionTypeField
request_stop_condition: MoveStopConditionField
acceleration: utils.Int32Field


@dataclass(eq=False)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -324,6 +324,8 @@ def _get_tip_action_motor_message(
velocity=self._convert_velocity(
step.velocity_mm_sec, tip_interrupts_per_sec
),
action=PipetteTipActionTypeField(step.action),
request_stop_condition=MoveStopConditionField(step.stop_condition),
acceleration=Int32Field(
int(
(
Expand All @@ -335,8 +337,6 @@ def _get_tip_action_motor_message(
* (2**31)
)
),
action=PipetteTipActionTypeField(step.action),
request_stop_condition=MoveStopConditionField(step.stop_condition),
)
return TipActionRequest(payload=tip_action_payload)

Expand Down