-
Notifications
You must be signed in to change notification settings - Fork 626
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
Pressure fluctuation - HVAC #13673
Comments
FDS Source: Issue #13673. Remove uninitialized variable
I still do not have a solution to this. I'm working on it. |
Here is the simplest example of the problem
|
Jason -- if you have a chance, can you look at the simple case above. If you run with 1 MPI process, there is no excessive ringing in the pressure. Run with 4 MPI processes and it rings. There must be some bad feedback loop between the HVAC vent and the pressure solver, but I cannot pin point it. The obstructions are not thin and they are not at mesh boundaries, which are the usual suspects with this kind of issue. |
In the case below, if I remove the fifth mesh (an empty disconnected box that does nothing), the case works without a ringing pressure. If I leave it, I see the ringing. My first guess is that somewhere there is a call to a routine with
|
Just wrapped up three weeks of travel. Haven't had time to delve into this yet. |
I think this has to do with pressure zones, processes, meshes, and connected zones. I think the fix should be easy, but I have to test it. |
FDS Source: Issue #13673. Fix bug in CONNECTED_ZONES
This change appears to have solved the original problem. There was a bug in the logic to indicate that two pressure zones had combined into one or split into two. |
While looking into the hvac routines I found a bug in handling merging pressure zones. It didn't impact this specific case but will make a difference in cases where two merging zones each have HVAC VENTs. |
i confirm the connected zones fix solves the issue |
This is a great news! Thank you very much for investigating it. If I want to test it, is this a correct version?: FDS-6.9.1-1275-g421fce2 2024-Nov-09 09:28 https://github.com/firemodels/test_bundles/releases/tag/FDS_TEST |
It will be in the next test release. |
We release a new test bundle everyday, assuming that our overnight tests are successful. So check back tomorrow. |
I have downloaded the test release, windows version and ran the original case. fds_local -p 8 -o 1 test.fds The diverging fluctuation issue is solved. Thank you! However the compartment pressure unexpectedly goes to positive 100 Pa at 7 s, when the room door opens. I have repeated the run with -15Pa boundary. The issue was still there, compartment pressure being 7 Pa. Would this be related the original issue or is it a separate one? Furthermore, not an issue but an observation; When I tried to run the Linux version, I have received the error |
we will have to take a look. probably related in some manner. glibc 2.34 was released in 2021. It is part of your linux distribution. |
Does that HVAC vent allow backflow? |
Yes. |
Something isn't right. Will have to look into it. |
I tired modifying the simple tunnel_4 case to have a similar sequence of door openings and closings but didn't see any strange behavior there. If I add duct velocity to the original case it seems to be correct. Nothing in the duct until 3 s when there is a path open from the -100 Pa VENT to the louver. At 4 s the inner door closes and the HVAC vent oscailltes for a short bit trying to equailize the pressure which is initially negative on the inside. At 6 s the outer door closes and a 7 s the inner door opens, but during this time HVAC flows are near zero. No where near enough to cause that jump in pressure. |
I plotted the This looks correct, in that the pressure gauges that are 2 m off the floor start and finish at about 101325 - 1.2 x 9.8 x 2 = 101301 Pa and the gauge that is 5.2 m off the floor begins and ends at 101325 - 1.2 x 9.8 x 5.2 = 101264 Pa I think the "problem" here is that the As for the oscillations---they are somewhat fictitious, but magically superimposing a solid obstruction in a flowing fluid instantaneously is also fictitious. |
I wrote some debug in the HVAC BC routine for the first wall cell evlauted for each node: WRITE(666,) 'PRE IOR:',NODE_INDEX,II,JJ,KK,B1%PRESSURE_ZONE,IOR,TMP(II,JJ,KK),RHO(II,JJ,KK) At the timestep after than 7 s door change: PRE IOR: 2 4 26 3 2 -2 293.193958049563 1.19501738389287 The internal node is still zone 2 and the external node is still zone 0. Same pressure, no flow. HVAC appears to be correct for this case and just the output issue noted in the prior post. |
Discussed in #13629
Originally posted by Er9y714 October 25, 2024
Hi,
I am posting this to discussions instead of issues because a similar issue was discussed before (#11951) and the behaviour was deemed normal. So this time I would like to share an another setup (DUCT with HVAC) and ask your opinion. Is this normal, if yes, how can I prevent it? Thank you.
I have observed a pressure issue in a complex simulation I am working on. In short, a room is connected to outside via a HVAC duct. When all doors are closed, the pressure in the room with duct does not drop to 0 Pa as I would expect.
I have tried to simplify the case as much as possible. I have also tightened the tolerances to the level I cannot apply to the complete project file. Yet, the issue is still present.
Although it is not an option for large project file, I have tried running this small case with ULMAT as well. It was significantly slower and fluctuation was still there.
(Similar issue #11951) Based on the previous posts/issues, I am aware that closing a door suddenly is expected to create such fluctuations but I have not yet found an alternative door opening/closing method. Furthermore, the average of the pressure in the room when all doors are closed is positive and not reducing. With a duct of 1.5 m2 area, I would expect there is not sufficient air supply to balance this pressure fluctuation but it appears not the case.
I have tried running this small case with ULMAT as well. It was significantly slower and fluctuation was still there. (This solver is not an option to use in complex project file)
Similar issue #11951
The text was updated successfully, but these errors were encountered: