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.

Sign up for GitHub

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

Jump to bottom

Enabling Dynamic Models - RO 0D Feature Enhancements #1499

Open
6 tasks
fahim831 opened this issue Oct 1, 2024 · 1 comment
Open
6 tasks

Enabling Dynamic Models - RO 0D Feature Enhancements #1499

fahim831 opened this issue Oct 1, 2024 · 1 comment
Assignees
@fahim831
Labels
Priority:Normal Normal Priority Issue or PR

Comments

@fahim831
Copy link
Contributor

fahim831 commented Oct 1, 2024 •
edited by adam-a-a
Loading

  • Implement material accumulation for permeate side of membrane in 0D
  • Modify costing block/functionality to handle dynamics
  • Add dynamics to 1dRO
  • Decide on/implement solution for dynamics for RO0D in simplest configuration, which excludes length/width construction
  • Decide on solution for unsteady momentum balance
  • add documentation for dynamic RO
@ksbeattie ksbeattie added the Priority:Normal Normal Priority Issue or PR label Oct 3, 2024
@fahim831
Copy link
Contributor Author

Dynamic models that utilize mass holdup terms are not yet fully worked out on IDAES due to elevated index issues. This needs to be fixed at the ControlVolume0D class level so it will take a while for a solution. Until this is resolved, further enhancements to the dynamic RO model is paused as it will not solve (correctly) either way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fahim831 fahim831

Labels
Priority:Normal Normal Priority Issue or PR
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ksbeattie @fahim831