-
Notifications
You must be signed in to change notification settings - Fork 7
All the offseason changes put together. #183
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
Draft
Jetblackdragon
wants to merge
44
commits into
main
Choose a base branch
from
soloPT
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
fe6c579
Added start for solo controller
Jetblackdragon e261298
All controls for solo controller
Jetblackdragon 77b4bae
Merge branch 'main' of https://github.com/robototes/Reefscape2025 int…
Jetblackdragon d66fb27
spot
Jetblackdragon 8a941ba
drive pratice changes
RobototesProgrammers c5f6f8e
V2 Set up for solo controls
Jetblackdragon e96486b
Override on driving when sticks are press down
Jetblackdragon d534668
dpot
Jetblackdragon fc7533b
LEDs for solo mode
TrisDooley 460a687
Drive practice changes
RobototesProgrammers 67b5733
Drive pracitce changes
RobototesProgrammers 94dc7fc
Rebump up current limits
Jetblackdragon 7d21c83
NO_GAME_PIECE mode actually works
Jetblackdragon e4a648d
Drive c
RobototesProgrammers b2d7d37
Code Clean up
Jetblackdragon a374024
New command that in theory changes the algae intake command based on …
Jetblackdragon 13299e7
added booleanInIntake
Jetblackdragon 3277113
Supercycling working
Jetblackdragon 7f8c421
Change so auto evelator movements get canceled when you have a game p…
TrisDooley 9ce9866
oops i did withtimeout instead of until
Jetblackdragon 311ce02
spot
Jetblackdragon 49cb5bc
spotspot
Jetblackdragon 4503b62
quick fix
Jetblackdragon 7c4aa9c
spot 😭
Jetblackdragon 8260fb2
ground intake sets intake mode to coral
Jetblackdragon c382db6
command name correction
Jetblackdragon b91070c
AutoAlgaeHeights Working
RobototesProgrammers d80af1a
hold voltage out until retract
RobototesProgrammers 0d78e78
Code cleanup
Jetblackdragon be66692
Merge branch 'cyclesuper' of https://github.com/robototes/Reefscape20…
Jetblackdragon 0b3c6f6
merge conflict
Jetblackdragon b64b691
spotttt
Jetblackdragon 7c30820
Merge branch 'cyclesuper' of https://github.com/robototes/Reefscape20…
Jetblackdragon d015c10
Prescore fix
Jetblackdragon 5fb07b7
Merge branch 'main' of https://github.com/robototes/Reefscape2025 int…
Jetblackdragon 4a686be
spot
Jetblackdragon 9e23717
More code clean up
Jetblackdragon 5293b44
s
Jetblackdragon c9f4263
holdGroundIntakeOut Fix
Jetblackdragon c9910b4
Fix for prescore bug
Jetblackdragon 03d1a52
Fixed barge score not finishing
Jetblackdragon 1cb71aa
pre score fix attempt #???
Jetblackdragon 4bdd1ea
controller connceted bug
Jetblackdragon 3564465
super cycle fix probably
Jetblackdragon File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this will result in
soloCC
only being set when the robot code starts running, so changing to or from soloCC will require a restart of the robot code. Consider instead making this a boolean class member variable and then updating it ever second or so based on whether the solo controller is connected (via adding a newupdate
method in this class thatRobot.robotPeriodic()
calls, and have theupdate
method do the soloCC update every second or so).Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it worked fine at drive practice we can check again at the meeting on monday
-Aarya (I forgot what account im on)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you think it actually matter is we just call soloController.isConnected() instead of soloCC for the drive request (changing it back to what is was before)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking at what isConnected does, it's similar to what driverController.getLeftX() does by looking at DriverStation.getStickAxis() vs DriverStation.isJoystickConnected
I wouldn't go back to the original change which did
.withVelocityX(soloController.isConnected() ? getSoloDriveX() : getDriveX())
, since it ends up reading from the HID four times (three times to check if the controller is connected & once to get that axis value), but instead have a class variable that's updated every loop to cache the result ofsoloController.isConnected()
and then switch off of that:.withVelocityX(soloCC ? getSoloDriveX() : getDriveX())
. To ensure the commands are run with the most up-to-date value, I'd sequence the new Command class periodic update inRobot.robotPeriodic
to happen before the call toCommandScheduler.getInstance().run();