Opened 4 years ago

Last modified 6 months ago

#1791 assigned Defect

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 4 years ago.
scale_factor_interp.pdf (156.6 KB) - added by jchanut 13 months ago.

Download all attachments as: .zip

Change History (16)

Changed 4 years ago by jchanut

comment:1 Changed 4 years ago by nicolasmartin

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


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

comment:2 Changed 3 years ago by nemo

  • Keywords VVL added; vvl removed

comment:3 Changed 3 years ago by clevy

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

comment:4 Changed 3 years ago by nicolasmartin

  • Type changed from Bug to Enhancement

comment:5 Changed 3 years ago by nemo

  • Keywords factors removed

comment:6 Changed 3 years ago by nemo

  • Keywords release-3.6* added

comment:7 Changed 3 years ago by nemo

  • Keywords release-3.6* removed

comment:8 Changed 16 months ago by jchanut

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

Changed 13 months ago by jchanut

comment:9 Changed 13 months ago by jchanut

In 10999:

Fix for ticket #1791

comment:10 Changed 13 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 13 months ago by jchanut

In 11012:

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

comment:12 Changed 10 months ago by jchanut

In 11422:

#1791, merge with trunk

comment:13 Changed 9 months ago by jchanut

  • Component changed from OCE to DOM

comment:14 Changed 6 months ago by jchanut

  • Type changed from Enhancement to Defect
Note: See TracTickets for help on using tickets.