-
Notifications
You must be signed in to change notification settings - Fork 2
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
Repo chat #17
Comments
@gwarby LOL - my bad! |
@gwarby One thing that kept me from tossing this code in sooner, is that it doesn't include a lot of really good ideas you've dev'ed in the branches. But unfortunately much of your code hasn't been proven & fine-tuned on the practice field, so the working competition code (in the new branch I mention above) is probably what we need to run - at least in the Sat 12/16/23 meet. Notable to-do's:
|
@lcdnbl I'll create a branch off of yours that implements my time saves and the new arm limiter, along with blueBackPixel picking up on init. Regardless of how much of that we actually achieve tomorrow night, I'll leave all the branches separated. Hopefully that will prevent any bad code needing to be run 12/16/23. |
@gwarby I checked in tho code on the robot after the meet, merged into the new competition stable branch, and deleted 'the other' branch. As far as I can tell, I didn't clobber any pending/candidate/untested time saving code. ...and maybe we should just rename this issue "repo chat" or some such thing. |
@gwarby I compared RedFrontPixelMatch10Config to RedFrontPixel, because I was concerned that we tweaked RedFrontPixel after making the copy. But, other than the 3.5 second delay(s) & the name change, they match. |
@gwarby Today's bot spot practice code is in the common_auto_functions branch (which we should probably merge onto/over the competition ready branch) |
@lcdnbl I don't know if we ever laid down a time for tonight, but Harry and I have swim until 6. I would guess any time after 6:15 would work. |
@gwarby @Harrison879 |
@gwarby if you want to work on the code offline I would suggest:
|
@lcdnbl I would be most grateful if you were to upload the updated code from yesterday. Please and thank you.
The text was updated successfully, but these errors were encountered: