Here's (change 820b68b) is an example implementation of configurable ButtonBindings that would allow us to express the differences, say, between main robot, testplatform 1, etc. From RobotContainer's POV, this seems to have very little negative impact. The config behavior is currently in a combination of Constants.OI and ButtonFactory.
I've done this work in a separate branch so it's really easy for me to punt. You can express the intent to punt by simply canceling this pull request.
Alternatively if this capability might be useful go ahead and accept the pull request.
Seems pretty harmless to me, but you never know. I have not had the opportunity to validate on a real device.
Here's (change 820b68b) is an example implementation of configurable ButtonBindings that would allow us to express the differences, say, between main robot, testplatform 1, etc. From RobotContainer's POV, this seems to have very little negative impact. The config behavior is currently in a combination of Constants.OI and ButtonFactory.
I've done this work in a separate branch so it's really easy for me to punt. You can express the intent to punt by simply canceling this pull request.
Alternatively if this capability might be useful go ahead and accept the pull request.
Seems pretty harmless to me, but you never know. I have not had the opportunity to validate on a real device.