-
Notifications
You must be signed in to change notification settings - Fork 0
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
James Auto Routines #10
Draft
jwaters3457
wants to merge
25
commits into
ANJ_MCP_variant
Choose a base branch
from
James_AutoRoutines
base: ANJ_MCP_variant
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
Conversation
This file contains 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
still figuring out how the org.json stuff works
^its just a string. i added the code for getting json from limelight/networktables into the jsonparser because i had nowhere else to put it. it will be moved later.
Co-authored-by: JPWaters111 <[email protected]>
Tech layoffs reached 150000 by now. Looks like our parsing thingamajig here is the next thing out.
I figured out how to use the org.json.simple library! I made a few things, all of which funnel into the final main "getDoubleProperty()" method That method is public and *automagically* fetches info about the currently tracked object for you. The subsystem constantly keeps parsing new up-to-date JSON and the method uses the most newly-parsed JSON each time so the numbers you get will always be current. If you want more details talk to me! -Drew M.
1. I tried to debug the code, it was really hard. I hope what I did was acceptable. We'll see... 2. good thing git saves your revisions because we will probably need to go back a few.
self-explanatory
Created Step to intake and outtake during auto
The balancing part needs to merge with Shane's code to function
Need to tune during robot testing
Updated base of PR to make it more readable |
fruzyna
reviewed
Feb 8, 2023
src/main/java/org/wildstang/year2023/auto/Steps/IntakeStep.java
Outdated
Show resolved
Hide resolved
src/main/java/org/wildstang/year2023/subsystems/targeting/AimHelper.java
Show resolved
Hide resolved
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Subsystem: Auto Programs
Description:
Created the auto program and steps for a one score, one grab, and balance program. Added necessary functions into master control program and intake to work.
Needs to change enum values, adjust intake parameters, and merge with Shane's balance program before code is functional.
Also assumes that we're using Aaron's intake code and Nathan, Jonah, and Shane's Master Control Program.
Tested:
No
Collaborators:
Before submitting: