-
Notifications
You must be signed in to change notification settings - Fork 640
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
Lattimer Tilted Wall Val Guide #14297
Comments
Thanks for the reminder. These were in flux when we were dealing with the GEOM radiation issues. I think we want to benchmark with them going forward. I can add the devices and get them added to dataplot. |
OK, thanks @johodges |
Quick update on this. I have updated input files, dataplot, and the guide for these. Still waiting on the runs to finish before adding to out and sending the PRs. I should have results in the next day or 2 so we can get these pulled in to master. One thing I noticed is that we do see artifacts at the mesh boundary. |
Are these devices that are defined in inter-mesh boundaries? |
Yep, these are line devices defined along a rotated GEOM that crosses a mesh boundary. The input file for the bottom right figure is here: |
I don't see an issue in the BNDF of Gauge heat flux, two devices for GHF at z=0.2m are not getting updated. Using these lines only for devices 23-26 of the line:
I see:
where GFH2 and GFH3 don't get updated. Those two are not paired to a CFACE, need to understand why. |
@rmcdermo I am assuming we want to wait on merging these in until after the minor release bundle is built. Let me know if you want them pulled in before. |
There is an assumption when pairing devices that the search for solid CFACES will happen only if a wall cell has not been found (IW==0). Turns out here a wall cell has been found, but is of type INTERPOLATED. We need an extra test for pairing DV to CFACES next to external wall cells. I'm testing a solution for this.
|
I ran all the Lattimer tilted wall cases. Was about to move the new results to the .out folder. But that folder currently has a single pair of files (devc and line) for the 20 kW 0 degree case. The current FDS files do not create a line file (only a devc) and there are many more input files than just the 20 kW 0.
What files need to be put in out? Do the fds inputs need to be edited to create a line file? If we are not using all the various inputs file should we remove them from the repo so we are not spending time running them?
The text was updated successfully, but these errors were encountered: