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.

Sign up for GitHub

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

Jump to bottom

ControlGroup cleanup. #1128

Open
robfalck opened this issue Nov 6, 2024 · 0 comments
Open

ControlGroup cleanup. #1128

robfalck opened this issue Nov 6, 2024 · 0 comments
Labels
Refactor/Cleanup Removal of technical debt.

Comments

@robfalck
Copy link
Contributor

robfalck commented Nov 6, 2024

Task Description

Previously ControlGroup existed to house separate ControlInterpComp and PolynomialControlInterpComp components. Since these have been united, there is no longer a reason for the ControlGroup to exist.

With the development of new Radau and GaussLobatto transcriptions that handle state continuity defects, there will no longer be a need for ContinuityComp for these transcriptions. ControlComp should handle the compute and constraining of control value and rate defects by itself so that we may remove the ContinuityComp from these transcriptions.
@robfalck robfalck assigned robfalck and unassigned robfalck Nov 6, 2024
@robfalck robfalck added the Refactor/Cleanup Removal of technical debt. label Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Refactor/Cleanup Removal of technical debt.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@robfalck