gavin971 / cvmix

Automatically exported from code.google.com/p/cvmix
0 stars 0 forks source link

Add option to use parabolic shape function #42

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Add a flag to cvmix_kpp_param_type and code to use parabolic shape function 
instead of simple shape function or matching derivatives at bottom of boundary 
layer [see Eq. (7.203b) in Section 7.6.4]

I think the best way to code this up will be to add another option to 
MatchTechnique (ParabolicShapes); once added I'll punt this back to GFDL to 
test against their current implementation.

Original issue reported on code.google.com by mike.lev...@gmail.com on 30 Jan 2014 at 9:52

GoogleCodeExporter commented 9 years ago

Original comment by mike.lev...@gmail.com on 30 Jan 2014 at 9:52

GoogleCodeExporter commented 9 years ago
This went in to r279 as outlined above, except the option is

MatchTechnique='SimpleParabolas'

Perhaps "MatchTechnique" isn't the best name for this option? I'll leave this 
ticket open until I hear from GFDL that it matches what they hacked in on their 
own.

Original comment by mike.lev...@gmail.com on 2 Feb 2014 at 8:45

GoogleCodeExporter commented 9 years ago
r280 is a second attempt after Steve pointed out that only the non-local term 
should use this shape function (gradient term should be the simple cubic). So I 
removed

MatchTechnique='SimpleParabolas'

in favor of

MatchTechnique='ParabolicNonLocal'

which doesn't affect the gradient term at all.

Original comment by mike.lev...@gmail.com on 3 Feb 2014 at 4:56

GoogleCodeExporter commented 9 years ago

Original comment by mike.lev...@gmail.com on 9 Feb 2014 at 6:42