Skip to content

(#81) Setup of TransferRoller subsystem#83

Merged
thetaback merged 16 commits intomainfrom
81-add-intake-roller
Mar 7, 2026
Merged

(#81) Setup of TransferRoller subsystem#83
thetaback merged 16 commits intomainfrom
81-add-intake-roller

Conversation

@thetaback
Copy link
Contributor

  • added Idle, TestMode, Spin, and DeJam states
  • subsystem pretty much the same as indexer
  • currently runs at the same time as hopper and indexer

- added Idle, TestMode, Spin, and DeJam states
- subsystem pretty much the same as indexer
- currently runs at the same time as hopper and indexer
@thetaback thetaback requested a review from aidnem February 28, 2026 23:49
Copy link
Contributor

@godmar godmar left a comment

Choose a reason for hiding this comment

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

See comments

@aidnem aidnem added integration These are PRs that are being integrated or are in the queue to be integrated hardware access Needs access to the physical robot/parts of it to be accomplished labels Mar 6, 2026
godmar added 3 commits March 6, 2026 08:57
I hand merged elastic_layouts/tuning.json by just including the tab for
the transfer roller.
- made transferRoller.coast public to allow access from coordination
  layer
- have coordination layer use coast() function
@godmar
Copy link
Contributor

godmar commented Mar 6, 2026

@thetaback I merged main into this and renamed transferRoller to transferroller.

aidnem
aidnem previously requested changes Mar 7, 2026
@thetaback thetaback requested a review from godmar March 7, 2026 17:46
Copy link
Contributor

@godmar godmar left a comment

Choose a reason for hiding this comment

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

LGTM

@thetaback thetaback dismissed aidnem’s stale review March 7, 2026 17:51

Aiden said it was ok to do this

@thetaback thetaback merged commit f7729f5 into main Mar 7, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hardware access Needs access to the physical robot/parts of it to be accomplished integration These are PRs that are being integrated or are in the queue to be integrated

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants