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.

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
wants to merge 25 commits into
base: ANJ_MCP_variant
Choose a base branch
from
Draft

Conversation

jwaters3457
Copy link
Contributor

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:

  • Review that your code meets our coding standards
  • Confirm there is a green ✔️ (not a red ❌) next to your last commit's date
  • Replace all text above with appropriate descriptions
  • Remove header and footer from this template
  • Add @smccrorie and @fruzyna as reviewers

foxler2010 and others added 19 commits January 17, 2023 20:30
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 <jwaters3457@users.noreply.github.com>
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
@jwaters3457 jwaters3457 marked this pull request as draft February 4, 2023 20:27
@jwaters3457 jwaters3457 marked this pull request as ready for review February 7, 2023 01:28
@jwaters3457 jwaters3457 marked this pull request as draft February 7, 2023 01:29
@fruzyna fruzyna changed the base branch from main to ANJ_MCP_variant February 8, 2023 00:21
@fruzyna
Copy link
Member

fruzyna commented Feb 8, 2023

Updated base of PR to make it more readable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants