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

A bug in the evacuation geometry #38

Closed
gforney opened this issue Jun 24, 2015 · 2 comments
Closed

A bug in the evacuation geometry #38

gforney opened this issue Jun 24, 2015 · 2 comments

Comments

@gforney
Copy link
Contributor

gforney commented Jun 24, 2015


Flow fields are not nice, if the walls are one grid cell thick
in the input file. They used to be fine, say, some month ago but
not any more. See the attached examples, one with 1 grid cell
thick walls and the other with 2 grid cell thick walls.
I did not yet do a test, where the walls are very thin, i.e.,
zero width: read.f90 should have 'thicken_obstructions=.true.'
for the evacuation flow grids. Probably, this case will also
have the same bug. Below is the source version:

 Compilation Date : May 27, 2007        
 Version          : 5_RC4+              

Ciao,
Timo Korhonen

Original issue reported on code.google.com by tkorhon1 on 2007-05-28 12:32:18


- _Attachment: [EvacFlowError.zip](https://storage.googleapis.com/google-code-attachments/fds-smv/issue-38/comment-0/EvacFlowError.zip)_
@gforney
Copy link
Contributor Author

gforney commented Jun 24, 2015

The problem was the 'sawtooth=.false.' default for the evacuation OBSTs.
Sawtooth=.false. does not work for obstacles that are only one grid cell thick. The
FDS5 fire calculation has the same problem, i.e., flow will go thorugh wall, if they
are defined as 'sawtooth=.false.' and are one grid cell thick.

Original issue reported on code.google.com by tkorhon1 on 2007-06-06 10:56:16

@gforney
Copy link
Contributor Author

gforney commented Jun 24, 2015

The evacuation geometry has 'sawtooth=.true.' now, so there is no
problems with 'sawtooth=.false.'. And Kevin also corrected the sawtooth
feature, so one could also have 'sawtooth=.false.' for evacuation
geometry, but this is not needed. The evacuation flow fields seem
to be fine even with the 'sawtooth=.true.' case. So, let's say that
this issue is 'verified'.

Timo

Original issue reported on code.google.com by tkorhon1 on 2007-09-24 13:25:35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant