Skip to content

Corner backup shot for defense#108

Open
themehdev wants to merge 5 commits intomainfrom
corner-backup-shot
Open

Corner backup shot for defense#108
themehdev wants to merge 5 commits intomainfrom
corner-backup-shot

Conversation

@themehdev
Copy link
Copy Markdown
Contributor

@themehdev themehdev commented Apr 4, 2026

We should test on robot then merge.
I have tested both sides in sim now, and it works very well.

there is this weird like giggle it does at the waypoint but idk why that is happening.
@themehdev themehdev added enhancement New feature or request should happen These pull requests and issues should be looked at and worked on promptly needs testing Needs to be tested on the robot before merging. labels Apr 4, 2026
@themehdev
Copy link
Copy Markdown
Contributor Author

themehdev commented Apr 7, 2026

It should be fixed now, and I still think its useful. I can change it to be only the outpost side because that side will be more accurate if you want. However, I still think that both sides are good to have against defense.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

There are a lot of weird github issues that make it seem like it is way more complicated than it is. I only added constants to the positions class, and got a rid of the ferryConstants classes that we were not using and only making things more confusing. I also indented some things correctly because otherwise I was unable to read which code was in what classes.

@themehdev
Copy link
Copy Markdown
Contributor Author

Also this is a cool flex to judges for autonomous award

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request needs testing Needs to be tested on the robot before merging. should happen These pull requests and issues should be looked at and worked on promptly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant