Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Comp Merge #164

Merged
merged 26 commits into from
Mar 8, 2024
Merged

Comp Merge #164

merged 26 commits into from
Mar 8, 2024

Conversation

ACat701
Copy link
Contributor

@ACat701 ACat701 commented Feb 22, 2024

AUTO TESTING:
-1. Finish adding event markers to the paths (Will be done before the meeting starts. its just intaking)

  1. Tune Autonomous pid using a 1m by 1m square
  2. Test WING Autos
    • We're mostly worrying about the zeroing of the pitch here
  3. Figure out how far down the centerline auto can start
  4. Test Centerline autos
  5. cry

@ACat701
Copy link
Contributor Author

ACat701 commented Feb 26, 2024

Code_QyoDLmahb6
@Alenguye582 The green ghost is the LEDs desired pose :D
ok I'm going to sleep now

@Alenguye582 Alenguye582 linked an issue Feb 26, 2024 that may be closed by this pull request
@ACat701
Copy link
Contributor Author

ACat701 commented Feb 26, 2024

I will focus on making auto paths sometime tonight if I can. Here's the auto list I think is realistic for phr. Most of the autonomous work has already been done (sim is awesome); its mostly getting vision now and then tuning the auto pid/paths on the real bot.

We also (and i cannot stress this enough) NEED to test driving on the opposing alliance. I'm almost confident that we need to flip the driver controls in some way.

WING AUTOS:

Start at Top of Subwoofer, score all 4 in a line into speaker
Start at Bottom of Subwoofer, score all 4 in a line into speaker

CENTERLINE "DASH" AUTOS:

Start in-between the top and bottom of subwoofer, immediately score in speaker then do a mad dash (top down)
Start as low down on the field as possible, immediately score in speaker then do a mad dash (bottom up)

OTHER AUTOS

Start at the top of sub, immediately score in the AMP
Start at some other location and just shoot into the speaker (the point will be to be out of the way. where should this be?)

Heres my elegant visualization of starting positions

auto starting pos

@ACat701
Copy link
Contributor Author

ACat701 commented Feb 26, 2024

@TaylerUva @Alenguye582 ^^

@ACat701 ACat701 changed the title Create Wing Auto Create Autos Feb 27, 2024
@ACat701 ACat701 added urgent Issue should be addressed immediately Untested This pull request has not been tested at all Tested in Sim This pull request has been tested, but not on the real robot and removed Untested This pull request has not been tested at all labels Feb 27, 2024
@ACat701
Copy link
Contributor Author

ACat701 commented Feb 28, 2024

Opening this pr not because it's ready, but because I will forget to open it when it is

@ACat701 ACat701 marked this pull request as ready for review February 28, 2024 22:07
@ACat701 ACat701 requested a review from a team as a code owner February 28, 2024 22:07
@ACat701 ACat701 enabled auto-merge (squash) February 28, 2024 23:46
@ACat701 ACat701 marked this pull request as draft March 5, 2024 03:11
auto-merge was automatically disabled March 5, 2024 03:11

Pull request was converted to draft

@ACat701
Copy link
Contributor Author

ACat701 commented Mar 5, 2024

i will clean up this pr before Thursday (it is SO messy rn)

@ACat701 ACat701 mentioned this pull request Mar 5, 2024
Copy link
Contributor Author

@ACat701 ACat701 left a comment

Choose a reason for hiding this comment

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

@TaylerUva Take your time getting to this comment/pr; i am incapable of taking a break

I glanced over this entire pr for bugs. It has 33 JAVA files and i think i almost died in the process. I left comments and made issues for major things i noticed; otherwise, I think its okay. This pr contains all of our testing from the past week or so so its obviously huge

@ACat701 ACat701 added Tested This pull request has been fully tested (on the real robot) and removed Tested in Sim This pull request has been tested, but not on the real robot labels Mar 5, 2024
@ACat701 ACat701 changed the title Create Autos Comp Merge Mar 8, 2024
@ACat701 ACat701 marked this pull request as ready for review March 8, 2024 02:52
@ACat701 ACat701 enabled auto-merge (squash) March 8, 2024 03:13
@ACat701 ACat701 merged commit 2f04649 into main Mar 8, 2024
1 check passed
@ACat701 ACat701 deleted the create-wing-auto- branch March 8, 2024 03:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tested This pull request has been fully tested (on the real robot) urgent Issue should be addressed immediately
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Get the Starting Pose for Auto for Placement LEDs
2 participants