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.

Sign up for GitHub

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

Jump to bottom

Auto Shuffleboard Chooser #84

Merged
merged 58 commits into from
Mar 22, 2024
Merged

Auto Shuffleboard Chooser #84

merged 58 commits into from
Mar 22, 2024

Conversation

kirbt
Copy link
Contributor

@kirbt kirbt commented Mar 9, 2024 •
edited
Loading

Shuffleboard Auto Path Filtering based off:

  • Start Position
  • Game Piece Count (excludes preloaded)
  • Vision

also added/tweaked a couple of autos cause why not

kirbt and others added 30 commits February 28, 2024 19:09
@kirbt
merg conflict
88fbc20
@kirbt
Merge branch 'main' of https://github.com/kirbt/Crescendo2024
2c121d7
@kirbt @stewluj
merge in
c7f6fde
@kirbt
Merge branch 'main' of https://github.com/kirbt/Crescendo2024
fd73875
@stewluj
Merge branch 'main' of https://github.com/kirbt/Crescendo2024
97b50eb
@kirbt
noob gaming
11dfa10
@kirbt
filter method
d757e84 
also registered more paths for shuffleboard
@kirbt
tweak
4d530b0 
tweak
@kirbt
taco
5a8c44f
@kirbt
chooser works yay
2feaf60
@kirbt
fumble
bd18f17
@kirbt
yay it work
fac7bcb
@kirbt
spotless boohoo hoo
0c733e1
@kirbt
added final
fc0b2dd
@kirbt
debug
56bb7f8
@kirbt
debug 2
bcee266
@kirbt
making autos beter
7b45642
@kirbt
start position within tolerance code
6ae0ef5
@kirbt
Merge branch 'main' of https://github.com/kirbt/Crescendo2024
87b1ed6
@kirbt
debug
1696d89
@kirbt
Merge branch 'main' into main
170ffb3
@kirbt
gulp
ad50df2
@kirbt
test
e04b7d8
@kirbt
Revert "test"
00e4238 
This reverts commit e04b7d8.
@kirbt
gulp
4288c1f
@kirbt
test
935416b
@kirbt
un noob
4d70760
@kirbt
Rename LowSpeakerCenterLineN5N4N3.auto to LowSpeakerCenterlineN5N4N3.…
ddbb2e5 
…auto
@kirbt
Rename LowSpeakerCenterLineN5N4.auto to LowSpeakerCenterlineN5N4.auto
6e328bb
@kirbt
grub
675e36b
TAKBS2412
TAKBS2412 previously approved these changes Mar 18, 2024
View reviewed changes
@kirbt kirbt closed this Mar 19, 2024
@kirbt kirbt reopened this Mar 19, 2024
TAKBS2412
TAKBS2412 reviewed Mar 19, 2024
View reviewed changes
Copy link
Contributor

@TAKBS2412 TAKBS2412 left a comment

Choose a reason for hiding this comment

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

I believe this breaks the auto field preview, at least when I tested it with Amp side subwoofer, vision, 2 game objects, and the steal auto.

@TAKBS2412 TAKBS2412 requested a review from KangarooKoala March 19, 2024 03:37
@kirbt
Copy link
Contributor Author

kirbt commented Mar 21, 2024

Updates:
Fixed Field Widget
Added Est. Time Widget

@TAKBS2412 TAKBS2412 linked an issue Mar 22, 2024 that may be closed by this pull request
AutoBuilder.buildAutoChooser() called twice at robotInit() #96
Closed
@TAKBS2412 TAKBS2412 dismissed bobtseattle’s stale review March 22, 2024 03:02

Need to merge for testing

@TAKBS2412 TAKBS2412 merged commit 508edfa into robototes:main Mar 22, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KangarooKoala KangarooKoala KangarooKoala left review comments

@TAKBS2412 TAKBS2412 TAKBS2412 approved these changes

@bobtseattle bobtseattle bobtseattle left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AutoBuilder.buildAutoChooser() called twice at robotInit()
5 participants
@kirbt @TAKBS2412 @bobtseattle @KangarooKoala @stewluj