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

Closure-specific temporary variables. #94

Closed
ali-ramadhan opened this issue Feb 28, 2019 · 3 comments
Closed

Closure-specific temporary variables. #94

ali-ramadhan opened this issue Feb 28, 2019 · 3 comments
Labels
abstractions 🎨 Whatever that means performance 🏍️ So we can get the wrong answer even faster

Comments

@ali-ramadhan
Copy link
Member

ali-ramadhan commented Feb 28, 2019

As discussed with @jm-c, I think another feature we will need is the ability to design closure-specific temporary variables.

For example, the implementation of Constant Smagorinsky or Anisotropic Minimum Dissipation will benefit from the ability to compute the eddy viscosity (at 4 locations in a cell --- 3 FaceFields and 1 CellField) and re-use it in calculating the contribution of the subgrid closure to the momentum 'source terms'.

Originally posted by @glwagner in https://github.com/ali-ramadhan/Oceananigans.jl/issues/73#issuecomment-467660286

@ali-ramadhan ali-ramadhan added performance 🏍️ So we can get the wrong answer even faster abstractions 🎨 Whatever that means labels Mar 2, 2019
@ali-ramadhan
Copy link
Member Author

ali-ramadhan commented May 22, 2019

I think we might be able to get away with recomputing, especially on the GPU, so closing this issue. Also not an actionable issue.

@johncmarshall54
Copy link

johncmarshall54 commented May 22, 2019 via email

@ali-ramadhan
Copy link
Member Author

@johncmarshall54 Agreed. Working on it as part of issue #161.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
abstractions 🎨 Whatever that means performance 🏍️ So we can get the wrong answer even faster
Projects
None yet
Development

No branches or pull requests

2 participants