Changes between Initial Version and Version 2 of Ticket #203


Ignore:
Timestamp:
Aug 24, 2018, 6:00:20 PM (6 years ago)
Author:
pesei
Comment:

Many thanks, we'll look at it!

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #203

    • Property Milestone changed from to FLEXPART 10
    • Property Component changed from FP physics/numerics to FP coding/compilation
    • Property Summary changed from inconsitency/bug in verttransform_gfs.f90 to inconsistency / bug in verttransform_gfs.f90
  • Ticket #203 – Description

    initial v2  
    33in FLEXPART 9.2 I stumbled upon what is most probably a bug, and it is still present in the FLEXPART git dev branch.
    44
    5 Comparing verttransform.f90 (now seems to be verttransform_ecmwf.f90) to verttransform_gfs.f90, around line 500, one notices pieces of code which probably should be identical, but they aren't:
     5Comparing `verttransform.f90` (now seems to be `verttransform_ecmwf.f90`) to `verttransform_gfs.f90`, around line 500, one notices pieces of code which probably should be identical, but they aren't:
    66
    77for ECMWF it looks like:
    88
    9       if (vv(nx/2-1,0,iz,n).lt.0.) then
     9 
     10{{{#!fortran
     11     if (vv(nx/2-1,0,iz,n).lt.0.) then
    1012        ddpol=atan(uu(nx/2-1,0,iz,n)/ &
    1113             vv(nx/2-1,0,iz,n))+xlonr
     
    1315        ddpol=pi+atan(uu(nx/2-1,0,iz,n)/ &
    1416             vv(nx/2-1,0,iz,n))+xlonr
     17}}}
    1518
    1619(+ + xlonr)
     
    1821whereas for GFS:
    1922
    20      if(vv(nx/2-1,0,iz,n).lt.0.) then
     23 
     24{{{#!fortran
     25    if(vv(nx/2-1,0,iz,n).lt.0.) then
    2126        ddpol=atan(uu(nx/2-1,0,iz,n)/vv(nx/2-1,0,iz,n))+xlonr
    2227     elseif (vv(nx/2-1,0,iz,n).gt.0.) then
    2328        ddpol=pi+atan(uu(nx/2-1,0,iz,n)/vv(nx/2-1,0,iz,n))-xlonr
     29}}}
    2430 
    2531(+ - xlonr)
hosted by ZAMG