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

Jump to bottom

Correct cross correlation of u and T in Waterloo Methanol #6714

Closed
rmcdermo opened this issue Jul 30, 2018 · 3 comments
Closed

Correct cross correlation of u and T in Waterloo Methanol #6714

rmcdermo opened this issue Jul 30, 2018 · 3 comments
Assignees
@mcgratta

Comments

@rmcdermo
Copy link
Contributor

screen shot 2018-07-30 at 11 20 41 am

@mcgratta I think the difference between the data and cross correlation with u*T here (left side of r=0) is just a matter of taking an absolute value.
mcgratta added a commit to mcgratta/fds that referenced this issue Jul 30, 2018
@mcgratta
FDS Source: Issue firemodels#6714. Add ABSOLUTE_VALUE for DEVC.
9d627ff
mcgratta added a commit that referenced this issue Jul 30, 2018
@mcgratta
Merge pull request #6715 from mcgratta/master
7463e92 
FDS Source: Issue #6714. Add ABSOLUTE_VALUE for DEVC.
@drjfloyd
Copy link
Contributor

drjfloyd commented Aug 2, 2018 •
edited

TIME_AVERAGED = FALSE writes the instantaneous value of the output, i.e. it is not averaged over DT_DEVC. ABSOLUTE_VALUE writes the absolute value of the output, i.e. if the DEVC output were -1 it would be written as 1.

@Er9y714
Copy link

Er9y714 commented Aug 2, 2018

Sorry, I realized that just after posting question.

mcgratta pushed a commit to mcgratta/fds that referenced this issue Aug 2, 2018
FDS Validation: Issue firemodels#6714. Use RADIAL VELOCITY in output
e89d87c
mcgratta added a commit that referenced this issue Aug 2, 2018
@mcgratta
Merge pull request #6743 from mcgratta/master
75e6d5e 
FDS Validation: Issue #6714. Use RADIAL VELOCITY in output
@mcgratta
Copy link
Contributor

mcgratta commented Aug 6, 2018

Have a look at the latest Validation Guide. I replaced the horizontal velocity u with a 'RADIAL VELOCITY' such that a positive value points away from the origin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mcgratta mcgratta

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@rmcdermo @mcgratta @drjfloyd @Er9y714