search

lanl / palm_lanl

LANL Contributions to PArallelized Large-eddy simulation Model (PALM)
2 stars 5 forks source link

Remove stratification term from AMD closure #72

Closed cbegeman closed 3 years ago

cbegeman commented 4 years ago

The stratification term in AMD closure is now off by default. See Issue https://github.com/xylar/palm_les_lanl/issues/69 for the reasoning behind this change.

Another minor change included in this PR is a namelist option for minimum timestep dt_min; once the minimum timestep is reached the simulation is terminated. This functionality was previously in place except the minimum timestep was set to 1e-5 * maximum timestep.

cbegeman commented 3 years ago

@xylar and @vanroekel I should have flagged this PR a while ago; sorry about that. I would like to get these changes in and submit this version for the software release (EIDR) because this is the branch I used for the runs in the manuscript I'm preparing.

xylar commented 3 years ago

@cbegeman, as far as I'm concerned, you can go ahead and merge as soon as @vanroekel is happy.