Conversation
e-bauman
left a comment
There was a problem hiding this comment.
main thing is to move the configs into constants.py
subsystems/Intake / Hopper/Intake.py
Outdated
|
|
||
| def zero_pivot(self): | ||
| self.pivot_angle = ( | ||
| (self.encoder.get_absolute_position().value)) |
There was a problem hiding this comment.
prettty sure phoenix automatically zeroes if you set up the config properly
e-bauman
left a comment
There was a problem hiding this comment.
looks pretty good, I had a few small notes. please also add docstrings, then you can start working on commands
subsystems/Intake / Hopper/Intake.py
Outdated
| self.pivot_motor.set_position(self.target_angle) | ||
|
|
||
| def update_table(self): | ||
| self.zeroedpub.set(self.pivot_motor_zeroed) |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
wait when does the intake get zeroed then if phoenix zeros it automatically in configs..............
how puzzling.....
There was a problem hiding this comment.
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
subsystems/intake/subsystem.py
Outdated
| 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) |
There was a problem hiding this comment.
look into auto zeroing in the documentation
i sincerely DOUBT this works
draft for intake subsystem