idaes-pse icon indicating copy to clipboard operation
idaes-pse copied to clipboard

Published 20 hours ago • verified publisher icon IDAES

Update uncertainty propagation toolbox to use sens.py in Pyomo

Open jialuw96 opened this issue 4 years ago • 3 comments

Challenge / problem:

  • sens.py was copied from Pyomo, and duplicated in IDAES uncertainty toolbox to get it work
  • sens.py has since been updated in Pyomo to improve code clarity and style
  • Two separate versions of sens.py is confusing

Proposal: remove sens.py in IDAES, update uncertainty toolbox to use sens.py in Pyomo

Tasks:

  • [ ] get_dsdp() in Pyomo sens.py assumes the Pyomo model has already been solved, whereas the version in IDAES solves the model

jialuw96 avatar Sep 23 '21 15:09 jialuw96

@jialuw96, is this something you will do?

ksbeattie avatar Sep 23 '21 18:09 ksbeattie

@adowling2 & @jialuw96, I'm pulling this off the release boards and adding the backlog label, in hopes that someone may eventually get to this.

ksbeattie avatar Jul 14 '22 19:07 ksbeattie

Sounds good. Looks like we'll need this for a milestone next project year so it will get done.

adowling2 avatar Jul 14 '22 19:07 adowling2