Skip to content

Commit 54931e6

Browse files
test: reorder unknowns consistently in linearization test
1 parent 1d544e1 commit 54931e6

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

test/downstream/linearize.jl

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,8 +125,9 @@ lsys = ModelingToolkit.reorder_unknowns(lsys0, unknowns(ssys), desired_order)
125125
@test lsys.C == [400 -4000]
126126
@test lsys.D == [4400 -4400]
127127

128-
lsyss, _ = ModelingToolkit.linearize_symbolic(pid, [reference.u, measurement.u],
128+
lsyss0, ssys2 = ModelingToolkit.linearize_symbolic(pid, [reference.u, measurement.u],
129129
[ctr_output.u])
130+
lsyss = ModelingToolkit.reorder_unknowns(lsyss0, unknowns(ssys2), desired_order)
130131

131132
@test ModelingToolkit.fixpoint_sub(
132133
lsyss.A, ModelingToolkit.defaults_and_guesses(pid)) == lsys.A
@@ -138,7 +139,7 @@ lsyss, _ = ModelingToolkit.linearize_symbolic(pid, [reference.u, measurement.u],
138139
lsyss.D, ModelingToolkit.defaults_and_guesses(pid)) == lsys.D
139140

140141
# Test with the reverse desired unknown order as well to verify that similarity transform and reoreder_unknowns really works
141-
lsys = ModelingToolkit.reorder_unknowns(lsys, unknowns(ssys), reverse(desired_order))
142+
lsys = ModelingToolkit.reorder_unknowns(lsys, desired_order, reverse(desired_order))
142143

143144
@test lsys.A == [-10 0; 0 0]
144145
@test lsys.B == [10 -10; 2 -2]

0 commit comments

Comments
 (0)