Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New Scaling Rollout - Running To Do #1565

Open
2 tasks
adam-a-a opened this issue Feb 7, 2025 · 1 comment
Open
2 tasks

New Scaling Rollout - Running To Do #1565

adam-a-a opened this issue Feb 7, 2025 · 1 comment
Assignees

Comments

@adam-a-a
Copy link
Contributor

adam-a-a commented Feb 7, 2025

New Scaler class approach should be rolled out across all IDAES models. However, we should also be careful that there is little to no loss in useful information; i.e., we should not recklessly replace calculate_scaling_factors routines for models with half-baked Scaler classes. Thus, we should have a way to benchmark current state of scaling vs. any revamped form of scaling through the new Scaler classes. We can take a shot at this by possibly making use of ScalingProfiler, and more likely, making use of IPOPTConvergenceAnalysis. The bare minimum so far would be to check condition numbers between existing scaling and any new scaling introduced.

  • Decide on initial, minimal set of scaling/robustness tests to apply to models
  • Select an initial model to apply new Scaler class to, along with tests that help benchmark "new" scaling against "old" scaling (calculate_scaling_factors); @dallan-keylogic suggests focusing on something that uses saponification since the property package scaler class already exists.

Coming "soon":
Running checklist of all models in IDAES that do not have Scaler classes yet. We'd then (predominantly) assign codeowners to select models. Right now, there will be too many models to list, and we should probably discuss further before we continue.

@adam-a-a
Copy link
Contributor Author

adam-a-a commented Feb 7, 2025

One more note: @MarcusHolly has already started rolling this out for some WaterTAP models (currently PRs), and we are still settling on robustness tests that we'd want to consider. Feel free to chime in @MarcusHolly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant