Skip to content
This repository has been archived by the owner on Mar 1, 2023. It is now read-only.

Add turbulence statistics diagnostics #1388

Merged
merged 2 commits into from
Aug 3, 2020
Merged

Add turbulence statistics diagnostics #1388

merged 2 commits into from
Aug 3, 2020

Conversation

slifer50
Copy link
Contributor

@slifer50 slifer50 commented Jul 26, 2020

Description

Defines a callback for calculating volumetrically averaged dimensionless kinetic energy and dissipation

I have built a module that allows for a callback to calculate volumetrically averaged dimensionless kinetic energy and kinetic energy dissipation. This will be useful as it replaces the tedious procedure of calculating these during post-processing. These represent useful insight into certain turbulence dominated cases like the Taylor-Green vortex.

  • Written and run all necessary tests with CLIMA by including tests/runtests.jl
  • Followed all necessary style guidelines and run julia .dev/climaformat.jl .
  • Updated the documentation to reflect changes from this PR.

For review by CLIMA developers

  • There are no open pull requests for this already
  • CLIMA developers with relevant expertise have been assigned to review this submission
  • The code conforms to the style guidelines and has consistent naming conventions. julia .dev/format.jl has been run in a separate commit.
  • This code does what it is technically intended to do (all numerics make sense physically and/or computationally)

@slifer50 slifer50 requested review from smarras79 and akshaysridhar and removed request for smarras79 July 26, 2020 15:54
@slifer50 slifer50 self-assigned this Jul 26, 2020
@slifer50
Copy link
Contributor Author

bors try

bors bot added a commit that referenced this pull request Jul 26, 2020
@bors
Copy link
Contributor

bors bot commented Jul 26, 2020

@kpamnany
Copy link
Contributor

This doesn't seem sufficiently general a solution to me.

@slifer50
Copy link
Contributor Author

slifer50 commented Jul 27, 2020

This doesn't seem sufficiently general a solution to me.

@kpamnany I would have liked to have this included in a diagnostics group but I need access to data from previous time steps and I'm not sure how to approach this from within the diagnostics framework.

@smarras79
Copy link
Contributor

This doesn't seem sufficiently general a solution to me.

@kpamnany how can this be made more general? turbulence diagnostics need to be part of the code; we should be able to turn them on and off as needed.
Can you please help @slifer50 make it general? thanks

@kpamnany kpamnany force-pushed the yt/turbulence_stats branch 2 times, most recently from 0e2ebf3 to 89b6236 Compare July 29, 2020 18:03
@kpamnany
Copy link
Contributor

@slifer50: I changed this into a diagnostics group. Please check out the output from the Taylor Green experiment (run with --diagnostics=default).

As another note, you probably need to change #1360 -- instead of adding nor into DiagnosticsGroup, you can follow the method used in this PR, here.

@kpamnany kpamnany changed the title Yt/turbulence stats Add turbulence statistics diagnostics Jul 29, 2020
@kpamnany kpamnany force-pushed the yt/turbulence_stats branch 2 times, most recently from cb19c2e to e22da78 Compare July 30, 2020 14:12
@slifer50
Copy link
Contributor Author

@kpamnany Everything looks good to me. Awesome work!

@kpamnany
Copy link
Contributor

bors r+

bors bot added a commit that referenced this pull request Jul 31, 2020
1388: Add turbulence statistics diagnostics r=kpamnany a=slifer50

# Description

Defines a callback for calculating volumetrically averaged dimensionless kinetic energy and dissipation



Co-authored-by: yassine <[email protected]>
@bors
Copy link
Contributor

bors bot commented Jul 31, 2020

Build failed:

Energy + Dissipation calculation.

Co-authored-by: K Pamnany <[email protected]>
@kpamnany
Copy link
Contributor

kpamnany commented Aug 3, 2020

bors r+

@bors
Copy link
Contributor

bors bot commented Aug 3, 2020

@bors bors bot merged commit a226f40 into master Aug 3, 2020
@bors bors bot deleted the yt/turbulence_stats branch August 3, 2020 22:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants