Skip to content

Commit 2627140

Browse files
authored
Remove new command based "What's changed" (#2287)
* Remove old command based stuff also fix some broken links I found * add redirect
1 parent 01d51ec commit 2627140

File tree

4 files changed

+3
-89
lines changed

4 files changed

+3
-89
lines changed

source/docs/software/commandbased/command-based-changes.rst

Lines changed: 0 additions & 83 deletions
This file was deleted.

source/docs/software/commandbased/cpp-command-discussion.rst

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ A Technical Discussion on C++ Commands
44

55
This article will help you understand the reasoning behind some of the decisions made in the 2020 command-based framework (such as the use of ``std::unique_ptr``, CRTP in the form of ``CommandHelper<Base, Derived>``, etc.). You do not need to understand the information within this article to use the command-based framework in your robot code.
66

7-
.. note:: The model was further changed in 2023, as described `below <#2023 Updates>`_.
7+
.. note:: The model was further changed in 2023, as described :ref:`below <docs/software/commandbased/cpp-command-discussion:2023 Updates>`.
88

99
Ownership Model
1010
---------------
@@ -187,7 +187,7 @@ After a few years in the new command-based framework, the recommended way to cre
187187

188188
A significant root cause of most pain points was commands being passed by value in a non-polymorphic way. This made object slicing mistakes rather easy, and changes in composition structure could propagate type changes throughout the codebase: for example, if a ``ParallelRaceGroup`` were changed to a ``ParallelDeadlineGroup``, those type changes would propagate through the codebase. Passing around the object as a ``Command`` (as done in Java) would result in object slicing.
189189

190-
Additionally, various decorators weren't supported in C++ due to reasons described `above <#Templating Decorators>`_. As long as decorators were rarely used and were mainly to reduce verbosity (where Java was more verbose than C++), this was less of a problem. Once heavy usage of decorators was recommended, this became more of an issue.
190+
Additionally, various decorators weren't supported in C++ due to reasons described :ref:`above <docs/software/commandbased/cpp-command-discussion:Templating Decorators>`. As long as decorators were rarely used and were mainly to reduce verbosity (where Java was more verbose than C++), this was less of a problem. Once heavy usage of decorators was recommended, this became more of an issue.
191191

192192
``CommandPtr``
193193
^^^^^^^^^^^^^^

source/docs/software/commandbased/index.rst

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
Command-Based Programming
22
=========================
33

4-
.. note:: Old (pre-2020) command-based is no longer available in 2023. Users should migrate to the new command-based framework below. Documentation for old command-based is available `here <https://docs.wpilib.org/en/2021/docs/software/old-commandbased/index.html>`_.
5-
64
This sequence of articles serves as an introduction to and reference for the WPILib command-based framework.
75

86
For a collection of example projects using the command-based framework, see :ref:`docs/software/examples-tutorials/wpilib-examples:Command-Based Examples`.
@@ -22,7 +20,6 @@ For a collection of example projects using the command-based framework, see :ref
2220
pid-subsystems-commands
2321
profile-subsystems-commands
2422
profilepid-subsystems-commands
25-
command-based-changes
2623

2724
Passing Functions As Parameters
2825
-------------------------------

source/redirects.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -279,4 +279,4 @@
279279
"docs/zero-to-robot/step-4/running-benchtop-test.rst" "docs/zero-to-robot/step-4/running-test-program.rst"
280280
"docs/zero-to-robot/step-1/how-to-wire-a-robot.rst" "docs/zero-to-robot/step-1/intro-to-frc-robot-wiring.rst"
281281
"docs/hardware/hardware-basics/wiring-pneumatics.rst" "docs/hardware/hardware-basics/wiring-pneumatics-pcm.rst"
282-
282+
"docs/software/commandbased/command-based-changes.rst" "docs/software/commandbased/index.rst"

0 commit comments

Comments
 (0)