-
Notifications
You must be signed in to change notification settings - Fork 63
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
Getting different velocities in versions 0.9.0 and 0.10.0 #625
Comments
Hello @dad616610 , |
One aspect that comes to my mind is the tolerance. If you change the value of tol_res and tol_m to smaller values, does that change anything? It is a little harder to find a good compromise for the tolerance of the mass flow change for both gas and liquid media. |
Hello @dlohmeier Thank you for your reply and a way to fix this I've added a pp.pipeflow(
net,
mode="hydraulics",
friction_model="colebrook",
tol_p=tol,
tol_v=tol,
tol_T=tol,
tol_res=tol,
tol_m=tol,
) The results didn't change (at all) |
What I noted, when I tried to reproduce: In the example 'fluid="water" ' is defined. To reproduce the given output, this must be changed to 'fluid="lgas" '. |
I found one important aspect of why the Reynolds numbers would not match as the temperature. In pandapipes 0.9.0 we made an error by using the fluid temperature for calculation, but when calculating the reynolds number based on norm velocity, you also need to use the norm density / norm temperature. If I use the norm density, I get the desired result.
For me, this all equals. Unfortunately, the error increases with differing temperatures from normal conditions, and probably also with decreasing norm factors (with a norm factor of 0.15, we are already quite far away from ideal gas behavior). This is why your example leads to such great differences. |
Consider the following example
Example
Result in 0.9.0
Result in 0.10.0
As you can see the velocity is drastically changed. Is this intentional? Could this PR #597 be a velocity changer?
I used velocity to calculate the Reynolds number by hand using this formula:
, where
T
is the initial temperatureThe formula above in Python
With version 0.9.0 the Reynolds numbers calculated by pandapipes and by hand matched. With version 0.10.0, however, the Reynolds numbers didn't match. The Reynolds numbers calculated by pandapipes are identical in versions 0.9.0 and 0.10.0
The text was updated successfully, but these errors were encountered: