firemodels / fds

Fire Dynamics Simulator
https://pages.nist.gov/fds-smv/
Other
670 stars 626 forks source link

FDS+evac dies on slice file creation #1300

Closed gforney closed 9 years ago

gforney commented 9 years ago
I'm developing an evacuation addition to an existing (and working) fds file. Per instructions
in the fds+evac manual, I attempted to create a pressure slice file to examine the
evacuation flow field. The pressure slice works (job runs to completion at T=1 s) if
I don't add the EVACUATION=.TRUE. If I add the EVACUATION=.TRUE. to the SLCF input,
the program dies with an access exception:

 Fire Dynamics Simulator

 Compilation Date : Fri, 29 Oct 2010

 Version: 5.5.3; MPI Disabled; OpenMP Disabled
 SVN Revision No. : 7031

 Job TITLE        : Office Elevator Evacuation
 Job ID string    : Evac

forrtl: severe (157): Program Exception - access violation
Image              PC                Routine            Line        Source

fds5_win_64.exe    00000001402B33AD  Unknown               Unknown  Unknown
fds5_win_64.exe    00000001402BB1ED  Unknown               Unknown  Unknown
fds5_win_64.exe    000000014031ECD5  Unknown               Unknown  Unknown
fds5_win_64.exe    00000001404A5CBC  Unknown               Unknown  Unknown
fds5_win_64.exe    000000014058DDDC  Unknown               Unknown  Unknown
fds5_win_64.exe    000000014056CE2B  Unknown               Unknown  Unknown
kernel32.dll       0000000076C5F56D  Unknown               Unknown  Unknown
ntdll.dll          0000000076E92CC1  Unknown               Unknown  Unknown

Original issue reported on code.google.com by cfastdev on 2011-02-09 17:31:03


gforney commented 9 years ago
Well, 

&SLCF PBZ = 1.000, QUANTITY = 'VELOCITY', VECTOR = .TRUE., EVACUATION=.TRUE. / 

should work. But it might be true that the pressure does not work.
It has been many years since I last tried to see the pressure slice
for evacuation meshes. I just see the flow vectors.

But I'll check why the error message appears with pressure slice.

TimoK

Original issue reported on code.google.com by tkorhon1 on 2011-02-10 08:10:46

gforney commented 9 years ago
The problem was that PRESSURE_ZONE array was not defined
for evacuation meshes. Now it is. Now the flow vectors
can be colored with the pressure (quantity = pressure and
vector = true). No pressure slices can be seen. The present
Smokeview shows (load/unload menu) only vector slices for
evacuation meshes and this will be so. Well, later I might
add slcf output, e.g., for agent density, but not now.

The SVN 7700 version has this "bug" fix. 

TODO: Update manual (actually, I'm totally rewriting it
for the "fds6" style user input for evacuation).

TODO: Add checks for the SLCF and evacuation meshes,
accept only VELOCITY and PRESSURE. Other outputs do
not need these checks, because they do not have the
optional keyword EVACUATION= true/false.

I'll change the status to fixed, I have added the SLCF
input check to my todo-list, so I'll remember it.

TimoK

Original issue reported on code.google.com by tkorhon1 on 2011-02-10 09:57:27

gforney commented 9 years ago
Thanks for not just replying that I was being stupid ... I was though.  I edited an
existing slice to get the evacuation slice and neglected to change the pressure to
velocity.  I did actually mean to use velocity so do whatever you think is appropriate
with the code.

Attribute this to new FDS+Evac user stupidity.

Original issue reported on code.google.com by cfastdev on 2011-02-10 15:34:50

gforney commented 9 years ago
Well, let's close this Issue. The only thing I could do
would be to add some user input checks, but that is in
my ToDo list for the evacuation inputs already.

TimoK

Original issue reported on code.google.com by tkorhon1 on 2011-03-24 15:40:25