Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

for consideration - don't move the shoulder at the end of the climb #106

Closed
wants to merge 1 commit into from

Conversation

dejabot
Copy link
Contributor

@dejabot dejabot commented Mar 23, 2024

Description

something to discuss and consider: Tony was expressing concern that it may be challenging for the BoxOp (Anthony) if a finger slips in the middle of climbing and the shoulder suddenly moves.

also tries to tighten up some logic for releasing the shoulder - some logs were showing shoulder context ownership release without ownership of the context.

How Has This Been Tested?

just for consideration - needs testing

  • Unit tests: [Add your description here]
  • Simulator testing: [Add your description here]
  • On-robot bench testing: [Add your description here]
  • On-robot field testing: [Add your description here]

@dejabot dejabot requested review from 1yd1a and qntmcube March 23, 2024 14:37
@dejabot dejabot closed this Mar 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant