I recently finished teaching an optimization-based controls course and found a but of small things in the code that needing fixing/improving:

• Parameters were not being passed properly to differentially flat systems.
• Basis functions derived fromBasisFamily were not callingBasisFamily.__init__.
• Changedconstraints keyword inoptimal.solve_ocp andflatsys.point_to_point totrajectory_constraints for more consistency (legacy keyword still works).
• Initial conditions for simulation of discrete time systems was being passed as a list instead of an ndarray, which caused some odd behavior and wasn't consistent with continuous time system usage.
• Added check for constraint types in optimal control problem setup (rather than when solving the optimal control problem).
• Fixed small inconsistency in the way discrete time was handled in setting up an I/O system for model predictive control (should be inherited from system, not passed separately).
• Unused keywords inOptimalControlProblem now generate an error (mentionedhere).
• To be consistent with the normal problem formulation, the discrete time optimal cost calculation no longer includes the endpoint (useterminal_cost instead).
• create_mpc_iosystem now re-computes the optimal controller when returning the output, which eliminates an unintended one-step delay in the MPC dynamics.
• Added unit tests.