Skip to content

Add even better autos for VAFAL#161

Merged
honzikschenk merged 12 commits intomainfrom
better-autos-for-VAFAL
Mar 19, 2024
Merged

Add even better autos for VAFAL#161
honzikschenk merged 12 commits intomainfrom
better-autos-for-VAFAL

Conversation

@aidnem
Copy link
Contributor

@aidnem aidnem commented Mar 11, 2024

Purpose
The purpose of this addition is to make better, faster, and higher scoring autos for Falls Church.

Auto Options

  • Pure 6 note: S3-W3-W2-W1-C1-C2
    This seemed to already be implemented.
    At the moment, this auto is over the time limit (16.47s when it needs to be 15). Either we must hyper-optimize the paths or just wait for faster autos to be tested so that we can just make the robot move faster.

  • 2 notes from centerline: S3-C5-C4
    Should work and be well within the time limit at around 11.20 seconds. However, it has yet to be tested on the actual robot.

  • 3 notes from centerline: S3-C5-C4-C3
    This auto is over the time limit, sitting at around 16.82 seconds. It could either be optimized by moving the shoot point further away from the speaker, or by using a faster set of constraints for the robot. However, both of these solutions require us to use the physical robot to test. Therefore, it is waiting on either:

    1. Test shooting from slightly outside the wing (shooting from the bottom side of the field is the same distance as a slightly over half-court shot on the side of the field that we have built).
    2. Test faster autos (if the robot can move faster, it should be able to make the auto without much difficulty).
  • Any 4 notes off the floor from any start position
    Not really sure about this one yet, and it is not implemented as of now.

@aidnem aidnem linked an issue Mar 11, 2024 that may be closed by this pull request
4 tasks
@honzikschenk honzikschenk marked this pull request as ready for review March 14, 2024 23:43
@honzikschenk honzikschenk marked this pull request as draft March 14, 2024 23:45
@jkleiber
Copy link
Collaborator

@redPlover we should load all our auto paths in the CommandSwerveDrivetrain constructor rather than lazy loading a path at auto init. We can maintain a HashMap of PathPlannerAutos based on the auto chooser keys and then send the relevant command with getAutoCommand at auto init. This will help with our delay problems because file loading takes a non-trivial length of time.

Also they just released a path planner update that has some warmup commands we can run in disabled mode. Let's try pulling those in as well

@honzikschenk honzikschenk marked this pull request as ready for review March 19, 2024 00:27
@honzikschenk honzikschenk enabled auto-merge (squash) March 19, 2024 00:28
Copy link
Collaborator

@jkleiber jkleiber left a comment

Choose a reason for hiding this comment

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

Ship it, I read all of this

@honzikschenk honzikschenk merged commit e8543f3 into main Mar 19, 2024
@honzikschenk honzikschenk deleted the better-autos-for-VAFAL branch March 19, 2024 01:55
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.

Add even better autos for VAFAL

4 participants