Skip to content

Feat/intake&hopper#2

Draft
theA-Train wants to merge 12 commits intodevfrom
feat/intake&hopper
Draft

Feat/intake&hopper#2
theA-Train wants to merge 12 commits intodevfrom
feat/intake&hopper

Conversation

@theA-Train
Copy link

draft for intake subsystem

Copy link
Contributor

@e-bauman e-bauman left a comment

Choose a reason for hiding this comment

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

main thing is to move the configs into constants.py


def zero_pivot(self):
self.pivot_angle = (
(self.encoder.get_absolute_position().value))
Copy link
Contributor

Choose a reason for hiding this comment

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

prettty sure phoenix automatically zeroes if you set up the config properly

Copy link
Author

Choose a reason for hiding this comment

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

?

Copy link
Contributor

@e-bauman e-bauman left a comment

Choose a reason for hiding this comment

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

looks pretty good, I had a few small notes. please also add docstrings, then you can start working on commands

self.pivot_motor.set_position(self.target_angle)

def update_table(self):
self.zeroedpub.set(self.pivot_motor_zeroed)
Copy link
Contributor

Choose a reason for hiding this comment

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

you probably don't need to constantly update this value, you can just set it one time when the intake is zeroed and then it will remain like that indefinitely

Copy link
Author

Choose a reason for hiding this comment

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

true.....

Copy link
Author

Choose a reason for hiding this comment

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

wait when does the intake get zeroed then if phoenix zeros it automatically in configs..............

how puzzling.....

Copy link
Contributor

Choose a reason for hiding this comment

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

I would look in the documentation to find out more info about the auto zeroing since I don't really know how it works tbh

return self.horizontal_motor.get_supply_current()

"""
def zero_pivot(self):
self.pivot_angle = (
(self.encoder.get_absolute_position().value))
pos = self.pivot_angle
self.pivot_motor.set_position(pos)
Copy link
Contributor

Choose a reason for hiding this comment

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

look into auto zeroing in the documentation

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.

2 participants