Split condensing and regularization into preparation and feedback #1034
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Condensing modules now provide
condense
- all condensing operationscondense_rhs
- condensing the right hand sidecondense_lhs
- condensing the left hand sideSimilarly, regularization modules now provide
regularize
- all operationsregularize_rhs
- operations on the right hand sideregularize_lhs
- operations on the left hand sideOnly if RTI is split into preparation and feedback, the
_rhs
/_lhs
routines are called.Otherwise, operations are done all together.
Addresses #1031
NOTE: using convexify with split RTI phase has now relatively overhead, see #1035
Splitting the computations for
convexify
more efficiently should be done in another PR.