-
Notifications
You must be signed in to change notification settings - Fork 0
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
[FEATURE] Vapor fraction for WaterStream components #449
Comments
Thanks for the issue ! I agree it is relevant to know what is the current vapor fraction, we'll add that soon :) |
I think we can't put that in the connectors, because of the stream connectors, and I don't think we can put equations in connectors, so it will have to be in base classes |
Ok, I did not think about that. As I said, I have only put them in the four WaterStream/Baseclasses so that I can find them back in the exchangers for instance and I have also coded them in the pipes (the component itself, not the partial one). But doing this way, we do not really use the advantage of inheritance. And I guess putting it in the very basic flowmodel from the Partial package would not be very clean since it can be used for media other than water (meaning we would need a medium distinction in it to actually activate the vapor fractions or something like this I guess). So if you find a compromise it would be nice, otherwise, it is not a big problem, just a useful feature. |
Agreed, it is a simple feature, but tricky to implement properly... |
What is your precise need ?
Having developed a water medium model, i felt it would be useful to have vapor fractions here and there to check more easily if we are still in the good phase.
What solution do you see ?
I manually added an x_in and an x_out in each WaterStream/BaseClasses and in a few other components.
Do you see any other alternative ?
On a second thought, I think it would even be even better and easier to put vapor fractions directly in the WaterStream/Connectors Inlet and Outlet.
Little worry
As we must implement an if condition to determine in which thermodynamic state we are to calculate the vapor fraction, I wonder if we should recommand not to fix these vapor fractions as inputs of the model for convergence reasons. I think they should only be considered as indicators.
The text was updated successfully, but these errors were encountered: