-
Notifications
You must be signed in to change notification settings - Fork 0
get game data and figure out if we can sore based on our field position and if its our turn based on our alliance #18
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
base: main
Are you sure you want to change the base?
Conversation
…on and if its our turn
|
so for the feed vs scoring thing I'm not exactly sure what you had in mind. Do we want feed and scoring to be triggered by same button and for the robot to choose based on if it can score and the operator can override? Or will they just be seperate things? So far i just have it so that scoring req requires it to be in the right place and for it to be our turn to score so we dont score at the wrong time. I guess we wouldn't want to feed if we are in the scoring place and its our turn so i can add that. |
|
nvm i have to fix the game data thing and use a timer |
|
will work on this tn |
|
yeah so i think eventually what we want to move towards is the driver pressing 1 button and then the robot decides what to do. i think that would mean scoreReq and feedReq get like combined into one shootReq. i haven't actually looked at your code yet but what i guess this would look like would be: ready/intake -(shootReq && our turn && in alliance zone)-> score and yeah shootReq && not our turn && in alliance zone shouldn't do anything |
spellingcat
left a comment
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.
ok i talked to cassie and basically she a) wants to be able to feed from everywhere and b) feeding should be a diff button from scoring which yknow is not what i had envisioned personally but it's fine lol. anyway just 1 thing about the get alliance thing
…e the operator will choose
spellingcat
left a comment
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.
make sure to merge main in as well. keep your definitions of feed + score req
No description provided.