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

Make HRRPUL work across meshes #12924

Closed
rmcdermo opened this issue May 13, 2024 · 11 comments
Closed

Make HRRPUL work across meshes #12924

rmcdermo opened this issue May 13, 2024 · 11 comments
Assignees

Comments

@rmcdermo
Copy link
Contributor

Currently, HRRPUL only integrates across the mesh where the DEVC lives. I think that at this point we need to make this multi-mesh compatible. It is important for diagnostic purposes that we are able to do this easily, and this usually means putting a line of DEVC in the middle of the burner and checking that SUM(HRRPUV*DZ) gives the correct HRR. And we are also wanting more and more to check the grid dependence of HRRPUL(Z) in our attempts to develop grid independent fire growth models.

Here is a simple test case. The burner is 1 m2 and has HRRPUA=100 kW/m2. The _hrr.csv file shows 100 kW. The _line.csv file will integrate to 100 kW if there is 1 mesh. It will integrate to 25 kW if I split the domain into 4 meshes.

test.fds.txt

@mcgratta mcgratta self-assigned this May 13, 2024
@mcgratta
Copy link
Contributor

I think this feature can be replicated by the more general approach where we can have a linear array of points, each of which uses SPATIAL_STATs. I'll see.

@mcgratta
Copy link
Contributor

&DEVC XB=0,0,0,0,0.05,1.95, POINTS=20, QUANTITY='HRRPUV', ID='hrrpul', DX=1, DY=1, DZ=0.05, SPATIAL_STATISTIC='VOLUME INTEGRAL' /

This yields the HRR in each horizontal slice. You can get it to print out hrrpul by using a CONVERSION_FACTOR of 0.1 and UNITS='kW/m'

@rmcdermo
Copy link
Contributor Author

Yep. Works great. Sorry, I did not see that in the user guide.

@mcgratta
Copy link
Contributor

The general methodology was put in after HRRPUL. We should probably remove HRRPUL, but we have to first replace it in all the V&V. I'll do that.

@mcgratta mcgratta reopened this May 14, 2024
mcgratta added a commit to mcgratta/fds that referenced this issue May 15, 2024
@Ranhill
Copy link

Ranhill commented May 27, 2024

Can you please explain a bit about the physical significance of conversion_factor being 0.1 here? I do understand when the conversion_factor is used. But why it is 0.1 here is not clear to me.

@mcgratta
Copy link
Contributor

@Ranhill What are you talking about?

@drjfloyd
Copy link
Contributor

I think he is referring to your post two weeks ago where you wrote:

This yields the HRR in each horizontal slice. You can get it to print out hrrpul by using a CONVERSION_FACTOR of 0.1 and UNITS='kW/m'

@mcgratta
Copy link
Contributor

OK, the CONVERSION_FACTOR is sometimes a convenient way to change units of the output. The 0.1 in this case is related to the particular grid size of this case.

@Ranhill
Copy link

Ranhill commented May 28, 2024

Thanks for the answer. Still not clear.

If we are multiplying HRR*grid_length (0.1) then we will get kW.m. Or, we take the conversion factor as 10 (reciprocal of 0.1m).

Otherwise, I think we should get HRRPUA (not HRR) as an output and then multiply it by 0.1(m).

HRRPUA*grid_length = kW/m

@mcgratta
Copy link
Contributor

You may be right. I probably have it backwards.

@mcgratta
Copy link
Contributor

mcgratta commented Jul 9, 2024

HRRPUL has been replaced by something like this

&DEVC XB=0.0,0.0,0.0,0.0,4.50,337.50, QUANTITY='HRRPUV', DX=72., DY=72., DZ=4.5, SPATIAL_STATISTIC='VOLUME INTEGRAL', POINTS=38, Z_ID='Height', ID='HRR' /

That is, at each POINT, we integrate HRRPUV over a volume that is bounded by x-72<x<x+72, and similar for the other coordinate directions. The result is an array of HRRs, not HRRPUL, but for the purpose of estimating a flame height, we use the PERCENTILE function to figure out that height at which 97% (or whatever) of the total energy lies.

@mcgratta mcgratta closed this as completed Aug 1, 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

4 participants