Skip to content

Feat/two score auto#87

Open
AbbeySieg wants to merge 67 commits intomasterfrom
feat/two-score-auto
Open

Feat/two score auto#87
AbbeySieg wants to merge 67 commits intomasterfrom
feat/two-score-auto

Conversation

@AbbeySieg
Copy link
Contributor

No description provided.

danielbrownmsm and others added 30 commits March 23, 2023 16:46
* code for new intake with wrist
still need to write commands for it though

* actually create hardware objects for the new intake

* update values from hardware in the software interface
* field oreinted scoring written, builds, untested

* field-oriented scoring tested, works

* left and right bumpers for pickup positions so we can pickup on both sides of the robot
* code for new intake with wrist
still need to write commands for it though

* actually create hardware objects for the new intake

* update values from hardware in the software interface

* wrote commands to move the wrist to a certain position, either just straight to a position or incrementing it

* bind some of the wrist commands we need

* wrist tilt encoder on shuffleboard

* clean up the intake commands and add some more OKC_CALL stuff

* add an offset to the tilt encoder

* fix intake parameters

* more okc checks
… slower, make either the arm or intake move first depending on the context to avoid breaking the robot, among other things
…he tilted station, not the one with the door) and of course tune some more septoints
danielbrownmsm and others added 27 commits March 30, 2023 13:04
…round anymore though, not without ADDING ANOTHER STUPID BUTTON which would probably be the d-pad but that didn't work out so well at shawnee so anyways
I love life so much I want to go and outlive all the other teams and laugh on their graves
… stuff because new intake YYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYAAAAAAAAAAAAAAAAAAYYYYY
Copy link
Collaborator

@danielbrownmsm danielbrownmsm left a comment

Choose a reason for hiding this comment

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

@noahzemlin double check this
@AbbeySieg fix the things I mentioned

id "cpp"
id "google-test-test-suite"
id "edu.wpi.first.GradleRIO" version "2023.4.2"
id "edu.wpi.first.GradleRIO" version "2023.4.3"
Copy link
Collaborator

Choose a reason for hiding this comment

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

good stuff

Copy link
Collaborator

Choose a reason for hiding this comment

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

we should not add all these to master

@@ -47,9 +40,6 @@ void Intake::Periodic() {
case Manual:
VOKC_CALL(this->ManualControl());
Copy link
Collaborator

Choose a reason for hiding this comment

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

we should probably just remove any kind of different manual/auto control

# units are in meters
x_disp = 0.216
y_disp = 0.416
y_disp = 0.216
Copy link
Collaborator

Choose a reason for hiding this comment

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

weird change but if it got better in drive practice

double intake_power_;

ControlMode mode_ = Auto;
ControlMode mode_ = Manual;
Copy link
Collaborator

Choose a reason for hiding this comment

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

see note above about control modes

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.

3 participants