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

BNDF XB control #13242

Closed
drjfloyd opened this issue Jul 29, 2024 · 2 comments
Closed

BNDF XB control #13242

drjfloyd opened this issue Jul 29, 2024 · 2 comments
Assignees

Comments

@drjfloyd
Copy link
Contributor

I have a case with a large number of meshes where in only some locations I want boundary output. The smokeview domain limit feature takes some extra processing time as smokeview has to figure out whether or not to load a specific mesh or output patch. On the FDS input file size, it is a complicated geometry and having to one-by-one figure out which OBST need to have BNDF_OBST set would take a bit of time. I could bulk select in Pyrosim but many OBST extend beyond the region where I care about output and having to break all those up for more fine grained control would take time.

I think it would be useful to either have XB on BNDF or something like a DUMP_BNDF flag on MESH. Given how we generate the list of OBST and VENT patches for the BNDF output and write them, XB might be tricky to implement. DUMP_BNDF on MESH should be pretty easy to do as we would just skip calling any BNDF setup or output routine if that flag was FALSE for the current MESH. I think the MESH approach would not impact Smokeview as it can already deal with loading a partial set of BNDF files in cases whjere not all bf files in the smv file are present.

@drjfloyd drjfloyd self-assigned this Jul 29, 2024
@drjfloyd
Copy link
Contributor Author

Recent smokeview changes now limit loading to meshes within or touched by an XB but still think would be useful to not even write the data if not needed to make file transfer faster from an HPC to a local drive.

drjfloyd added a commit to drjfloyd/fds that referenced this issue Aug 5, 2024
drjfloyd added a commit that referenced this issue Aug 5, 2024
FDS Source: add BNDF_MESH to MESH. Issue #13242.
drjfloyd added a commit to drjfloyd/fds that referenced this issue Aug 5, 2024
drjfloyd added a commit that referenced this issue Aug 5, 2024
FDS User Guide: Tweak BNDF_MESH text Issue #13242
@drjfloyd
Copy link
Contributor Author

drjfloyd commented Aug 5, 2024

Added BNDF_MESH (default T) to MESH. When set to F it does not output any BNDF files for that MESH.

@drjfloyd drjfloyd closed this as completed Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant