Opened 3 years ago

Last modified 4 weeks ago

#1791 assigned Enhancement

Vertical scale factor interpolation with key_vvl

Reported by: jchanut Owned by: greffray
Priority: normal Milestone:
Component: DOM Version: trunk
Severity: minor Keywords: VVL scale vertical
Cc: greffray

Description

Context

With time varying vertical coordinate (e.g. z-s star or tilde), it seems that the horizontal interpolation of vertical scale factors is wrong. Attached an example of top interface anomaly at velocity points (on expect something like the sea level anomaly) defined as the vertical sum of scale factors minus total depth at rest.

Analysis

Gurvan's aware.

Fix

One should at least ensure that top interface matches sea-level anomaly at u and v points.
Correct definition ot the bottom most interface is more tricky across steps.

Commit History (4)

ChangesetAuthorTimeChangeLog
11422jchanut2019-08-08T15:40:47+02:00

#1791, merge with trunk

11012jchanut2019-05-20T18:18:05+02:00

#1791, report domvvl change in VORTEX test case. Modification of domvvl there should be removed.

11002jchanut2019-05-20T14:53:58+02:00

vvl interp, #1791: 1) Add resting depth at F-points. 2) Rewrite dom_vvl_interpol with 2 aditionnal methods for horizontal interpolation at U-V-F points (nmet=0,1,2 in subroutine header). Set nmet=1, i.e. interpolate interfaces instead of scale factors anomaly. 3) Changes in the rest of the code: consider input array in dom_vvl_interpol at T point for F scale factors interpolation.

10999jchanut2019-05-20T12:24:00+02:00

Fix for ticket #1791

Attachments (2)

vvl_interp_issue.jpg (66.7 KB) - added by jchanut 3 years ago.
scale_factor_interp.pdf (156.6 KB) - added by jchanut 5 months ago.

Download all attachments as: .zip

Change History (15)

Changed 3 years ago by jchanut

comment:1 Changed 3 years ago by nicolasmartin

Looks a lot better when including the image directly in the ticket:


Last edited 3 years ago by nicolasmartin (previous) (diff)

comment:2 Changed 3 years ago by nemo

  • Keywords VVL added; vvl removed

comment:3 Changed 2 years ago by clevy

  • Cc greffray added
  • Owner changed from jchanut to greffray
  • Status changed from new to assigned

comment:4 Changed 2 years ago by nicolasmartin

  • Type changed from Bug to Enhancement

comment:5 Changed 23 months ago by nemo

  • Keywords factors removed

comment:6 Changed 23 months ago by nemo

  • Keywords release-3.6* added

comment:7 Changed 23 months ago by nemo

  • Keywords release-3.6* removed

comment:8 Changed 8 months ago by jchanut

  • Severity set to minor
  • Version changed from release-3.6 to trunk

Changed 5 months ago by jchanut

comment:9 Changed 5 months ago by jchanut

In 10999:

Fix for ticket #1791

comment:10 Changed 5 months ago by jchanut

In 11002:

vvl interp, #1791: 1) Add resting depth at F-points. 2) Rewrite dom_vvl_interpol with 2 aditionnal methods for horizontal interpolation at U-V-F points (nmet=0,1,2 in subroutine header). Set nmet=1, i.e. interpolate interfaces instead of scale factors anomaly. 3) Changes in the rest of the code: consider input array in dom_vvl_interpol at T point for F scale factors interpolation.

comment:11 Changed 5 months ago by jchanut

In 11012:

#1791, report domvvl change in VORTEX test case. Modification of domvvl there should be removed.

comment:12 Changed 2 months ago by jchanut

In 11422:

#1791, merge with trunk

comment:13 Changed 4 weeks ago by jchanut

  • Component changed from OCE to DOM
Note: See TracTickets for help on using tickets.