|
|
|
![]() |
|
|||||||
|
||||||||
![]() |
| Thread Tools | Rate Thread | Display Modes |
|
#16
|
|||
|
|||
|
Re: Is it required to use 2017 control system?
Quote:
Yes, the sections on installing were updated for 2017. My point is, I don't feel ScreenSteps does enough to explain how Command-Based changed so much with 2017. The core concepts are the same, but the implementation has changed quite a bit. |
|
#17
|
||||
|
||||
|
Re: Is it required to use 2017 control system?
Quote:
Quote:
Updating WPILib and reflashing everything is still a pain, but it's much less so than if we did things the WPILib way of doing them. |
|
#18
|
||||||
|
||||||
|
Re: Is it required to use 2017 control system?
I believe the significant changes occurred prior to the 2016 season, not the 2017 season. Can you explain the changes you found this year that are causing issues?
|
|
#19
|
|||
|
|||
|
Re: Is it required to use 2017 control system?
Quote:
) |
|
#20
|
|||
|
|||
|
Re: Is it required to use 2017 control system?
Here's the history on key command-based robot header files:
Command.h CommandGroup.h The change that would definitely break things at compile time is the namespace change. According to the commit message, it requires a one line change to fix. Even in a large robot codebase, it's probably a 10 minute job to fix. Easier if you have a shared header file throughout your codebase -- for many teams that would be robotmap.h. |
|
#21
|
|||
|
|||
|
Re: Is it required to use 2017 control system?
Quote:
We finally were able to make some headway by making a quickie project with RobotBuilder. What it generated looks WAY WAY different from past years and from what the project wizard in Eclipse makes. RobotMap was the biggest change. It's no longer a simple header file, but a class with static member variables. This is our team's fifth (or sixth) year using command based C++, so we certainly have some experience using it. I'm hoping we just accidentally reinstalled the 2016 plugins or something. That would explain a lot. Last edited by KPSch : 30-01-2017 at 17:44. |
![]() |
| Thread Tools | |
| Display Modes | Rate This Thread |
|
|