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

Develop #91

Merged
merged 19 commits into from
Jul 24, 2019
Merged

Develop #91

merged 19 commits into from
Jul 24, 2019

Conversation

boxofrox
Copy link
Contributor

Merge robot code from IRI competition and tag.

PhyXTGears-1720 and others added 19 commits July 9, 2019 21:19
STATUS: Tested on compbot.

- Robby S, Logan H-D
Status: not tested.

- Justin C
No consideration made for time delta.  Assuming constant delta.  This
may bite us.

STATUS: Tested on compbot.

- Logan H-D
Not sure what this is for.  Assuming it's used for the SparkMax
libraries.

- Justin C
Also updated smart dashboard profile on driver station.

- Justin C
Driver controls B button toggles mode.
Report mode on smart dashboard.

STATUS: untested.

- Justin C
Caleb requests:

1.  Top position as high as possible.
2.  Delay closure of grip for high position to ensure solid grip.
3.  Raise mid position.

- Justin C
Instead of toggling repeatedly while the button is pressed.

STATUS: Tested on compbot.

- Justin C
Climb arm is slower than air cylinders.  Give climb arm a better head
start.

STATUS: Tested on compbot.

- Justin C
Requested by Caleb.

STATUS: Tested on compbot.

- Justin C
Hatch mechanism falls past bumper perimeter during defense acruing
penalties.  Add rubber tubing spring to mechanism and use PID to hold
bottom position.

Compliance achieved with belt slip.

STATUS: Tested on compbot.

- Justin C
Requested by Cole.

- Justin C
- Reduce acceleration factor from 50% to 40%.

- Introduce a minimum acceleration to ensure desired velocity setpoint
  is reached.  This code doesn't do what I thought it would do.

STATUS: tested on compbot

- Justin C

Signed-off-by: Justin Charette <charetjc@gmail.com>
…vars-not-declared

Hotfix: update declarations of refactored variables
@boxofrox boxofrox merged commit 3447334 into master Jul 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants