Changes between Initial Version and Version 1 of Ticket #505


Ignore:
Timestamp:
04/26/11 19:41:18 (13 years ago)
Author:
arango
Comment:

Yes, you are correct! Good catch... The relative vorticity is a vector since it is the curl of velocity (Grad X V) which gives a vector. I intended to have the vertical component of relative vorticity and not the magnitude. The vertical component is more insightful. It also give us the direction (sign) of rotation.

Many thanks for reporting this...

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #505

    • Property ResolutionFixed
    • Property Status newclosed
  • Ticket #505 – Description

    initial v1  
    33I was very glad to see vorticity added to average file output. However, I believe there are a few issues with the way it is being calculated:
    44
    5 - The last term in pvor (line 378) should read -dVdz_pr*dRdx_pr (instead of -dVdz_pr*dRde_pr)
     5 * The last term for '''pvor''' ('''vorticity.F''', line 378) should read '''-dVdz_pr * dRdx_pr''' (instead of '''-dVdz_pr * dRde_pr''')
    66
    7 - It looks like rvor is being treated as a scalar, so that all three components are added together. Shouldn't this either be the magnitude of the relative vorticity, or just its vertical component (the first term)?
     7 * It looks like relative vorticity, '''rvor''', is being treated as a scalar, so that all three components are added together. Shouldn't this either be the magnitude of the relative vorticity, or just its vertical component (the first term)?
    88
    9 - I believe there's an extra d(pden)/d(z) in the preamble of the code, where the expressions for pvor and rvor are.
     9 * I believe there's an extra '''d(pden)/d(z)''' in the preamble of the code, where the expressions for '''pvor''' and '''rvor''' are.
    1010
    11 I might be missing something, of course. I've attached a diff file in case it is useful. 
     11I might be missing something, of course. I've attached a diff file in case it is useful.
    1212
    1313Best Regards,