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

Prebuilt Translator Blocks for Generic Property Package #616

Open
dallan-keylogic opened this issue Dec 1, 2021 · 0 comments
Open

Prebuilt Translator Blocks for Generic Property Package #616

dallan-keylogic opened this issue Dec 1, 2021 · 0 comments
Assignees
Labels
enhancement New feature or request Priority:Low Low Priority Issue or PR unit models Issues dealing with the unit model libraries

Comments

@dallan-keylogic
Copy link
Contributor

Adding entropy of mixing terms using logarithmic concentration variables has caused some difficulty with pressure_changers. Trace components in a stream (on the order of x_j=1E-16) result in equation degeneracy when the isentropic assumption is used. In principle, this can be dealt with by scaling, but using scaling typically implies that the user knows what components are going to be present in roundoff quantity ahead of time. If that's the case, they might as well remove the components from the property package altogether.

Translator blocks are general tools, but manually creating all the constraints to remove a single component is something of a burden on the user. If both sides of the translator are using the Generic Property Package, it shouldn't be too much effort to write the constraints automatically. There are three relatively simple cases that we could support:

  1. Adding/removing a component that is about to be generated/already been consumed in a single stream.
  2. Mixing several streams with different components together
  3. Changing state definitions
@dallan-keylogic dallan-keylogic added enhancement New feature or request unit models Issues dealing with the unit model libraries labels Dec 1, 2021
@ksbeattie ksbeattie added the Priority:Low Low Priority Issue or PR label Dec 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Priority:Low Low Priority Issue or PR unit models Issues dealing with the unit model libraries
Projects
None yet
Development

No branches or pull requests

2 participants