John Eslick
John Eslick
@blnicho or @jsiirola I'm going to fix this, but want to make sure I understand the requirements. I think the answer here is to move this optional constraint for the...
I fixed the PID controller. I couldn't find other instances in the IDAES-PSE repo. I think we can close this?
@blnicho, thanks. We can definitely break the initial condition calculation step out of the PETSc utility and use it as a more general method if it makes sense. At least...
I'll approve, but you'll have to run black on this thing to get the tests passing.
Related to issue IDAES/idaes-dev#340, deciding how to organize these new examples.
As a less disruptive alternative, we could add and initialized flag to the models. Maybe in the ProcessBlockData class. Give it an enumerated type like NOT_SUPPORTED, NOT_RUN, SUCCESS, FAIL. Not...
@andrewlee94, this is a really unpleasant suggestion, but I've wondering about using @Robbybp's context manager that allows you to temporarily fix certain variables or get submodels for initialization. I know...
I didn't read through everything, so I may have missed this, but have you considered scaling?
I'm super late, but IDAES scaling is slightly complicated. The way it works (very roughly) is that you specify scaling on a subset of variables. Usually like pressure, flow, sizes......
@andrewlee94 I talked to Miguel about this, and I think Pyomo does catch this on purpose and throws and exception figuring you are making an error trying to give Barron...