firemodels / fds

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

Importing smooth terrain into FDS #13075

Closed rakyuv closed 2 weeks ago

rakyuv commented 2 weeks ago

Is your feature request related to a problem? Please describe. I download SRTM30 dataset, and convert them into list of square or rectangular cylinders using &OBSTs, and use it as an input to FDS. And this creates a collection of cylinders with specified heights. However, the sharp edges from these cylinders would affect the flow physics.

Describe the solution you'd like The solution would be to create a smooth surface from the topography information.

Describe alternatives you've considered I tried to use qgis2fds to see the terrain files it generates. Presently, I am having errors with qgis2fds, and therefore I don't know how the files created by qgis2fds would look like.

rmcdermo commented 2 weeks ago

You are not giving us much to go on. What I gather is that you would like to treat cylinders as smooth geometry. This is possible with GEOM in the latest version in FDS. Please read the user manual.

rakyuv commented 2 weeks ago

Hi @rmcdermo! Thanks for your answer. Sorry for my unclear request. I was talking about this problem with sharp edges. Capture d’écran 2024-06-18 à 15 18 58

I referred to the 2D terrain elevation using GEOM in the latest User Guide. Is there anyway it works with FDS 6.7.9 ?

rmcdermo commented 2 weeks ago

Probably will not work with 6.7.9.

I think the main issue is you need to use the latest qgis2fds and you need to make sure you are exporting GEOM, not OBST.

Additionally, with GEOM, you will be forced to use a special pressure solver that has a limit on the number of cells per mesh (about 50,000), so you will need to use domain decomposition and run in parallel, most likely (I cannot see your mesh). You should also make sure your geometry extends beyond your computational domain.

rakyuv commented 2 weeks ago

I understand. I can limit the number of cells per mesh to less than 50000. I will find a way to make it work with the latest version of qgis2fds and export the terrain information as GEOM.

Thanks

emanuelegissi commented 2 weeks ago

If you are having issues with qgis2fds, please let me know by opening an issue here: https://github.com/firetools/qgis2fds/issues

Thank you, Emanuele

Il giorno mar 18 giu 2024 alle ore 16:14 Rakesh @.***> ha scritto:

I understand. I can limit the number of cells per mesh to less than 50000. I will find a way to make it work with the latest version of qgis2fds and export the terrain information as GEOM.

Thanks

— Reply to this email directly, view it on GitHub https://github.com/firemodels/fds/issues/13075#issuecomment-2176211504, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACXMVYZXTZ2CO5QC4QKCAK3ZIA6CTAVCNFSM6AAAAABJQAFRC2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZWGIYTCNJQGQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- "When old people used to wish each other peace, we didn't understand the meaning of this word. Now we really understand." [Natalia Balasynovych, mayor of Vasilkiv, Ukraine 2022]

rakyuv commented 2 weeks ago

Thank you @emanuelegissi. I reinstalled the latest version of qgis2fds and it works perfectly to import terrain to FDS input file. I will try to do this same without any gui, for the purpose of automation of my simulation. I'll keep you posted, if I get stuck somewhere.

emanuelegissi commented 2 weeks ago

You are welcome, okay

Il giorno mer 19 giu 2024 alle ore 11:11 Rakesh @.***> ha scritto:

Thank you @emanuelegissi https://github.com/emanuelegissi. I reinstalled the latest version of qgis2fds and it works perfectly to import terrain to FDS input file. I will try to do this same without any gui, for the purpose of automation of my simulation. I'll keep you posted, if I get stuck somewhere.

— Reply to this email directly, view it on GitHub https://github.com/firemodels/fds/issues/13075#issuecomment-2178169447, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACXMVY5GMQGSVQJL7Z7NT63ZIFDMTAVCNFSM6AAAAABJQAFRC2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZYGE3DSNBUG4 . You are receiving this because you were mentioned.Message ID: @.***>

-- "When old people used to wish each other peace, we didn't understand the meaning of this word. Now we really understand." [Natalia Balasynovych, mayor of Vasilkiv, Ukraine 2022]