search

lanl / palm_lanl

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

Bugfix for grid initialization #68

Closed cbegeman closed 4 years ago

cbegeman commented 4 years ago

Reverts a change made in https://github.com/xylar/palm_les_lanl/pull/59. topo flag is now always set at the bottom boundary.

katsmith133 commented 4 years ago

@cbegeman Alright, I ran both a test to make sure the non-AMD part of the code is the same and a test to make sure AMD is producing the same results it was before commit 266ccaf93e950b432e41a36188a67bf9de12faaf. Both look good to me, so I think this PR is ok to merge when you are ready. Thanks for this!

cbegeman commented 4 years ago

@katsmith133 Thanks for catching this and testing!