williamedwards / autompc

Automatic Tuning for Data-driven Model Predictive Control
BSD 3-Clause "New" or "Revised" License
54 stars 13 forks source link

0.2-dev transformers summed rather than sequenced #20

Open krishauser opened 1 year ago

krishauser commented 1 year ago

The current composition of transformed OCPs uses a SumTransformer, but this doesn't properly use the cost transformer after the bounds transformer (which doesn't touch the cost). Perhaps should be SequecneTransformer