Skip to content

Comments

Enable wave diagnostics for WW3#621

Merged
jedwards4b merged 1 commit intoESCOMP:mainfrom
alperaltuntas:add_ww3_hist
Jan 20, 2026
Merged

Enable wave diagnostics for WW3#621
jedwards4b merged 1 commit intoESCOMP:mainfrom
alperaltuntas:add_ww3_hist

Conversation

@alperaltuntas
Copy link
Member

@alperaltuntas alperaltuntas commented Dec 24, 2025

This PR enables WW3 diagnostics via CMEPS diagnostics module.

  • Introduce new wave diagnostics: HS, T01, T01M, THM
  • By default, enable the following WW3 cpl history fields:
    • Sw_Hs (signficant wave height)
    • Sw_t01 (mean wave period)
    • Sw_t0m1 (mean wave period)
    • Sw_thm ((mean wave dir)
    • Sw_lamult (langmuir multiplier(
    • Sw_ustokes (stokes drift in x dir)
    • Sw_vstokes. (stokes drift in y dir)
  • These fields are written as daily averages into monthly history files: ([CASENAME].cpl.hx.ww3.[DATESTAMP].nc):

Testing: aux_ww3, beta07. Status: b4b

To be evaluated in conjunction with: ESCOMP/WW3#38

- Introduce new diagnostics: T01, T01M, THM
- By default, enable the following daily diags in monthly files for WW3 cpl hist:
  - Sw_Hs:Sw_t01:Sw_t0m1:Sw_thm:Sw_lamult:Sw_ustokes:Sw_vstokes.
@alperaltuntas
Copy link
Member Author

Mentioning @gustavo-marques .

@jedwards4b jedwards4b merged commit 9991b7b into ESCOMP:main Jan 20, 2026
4 checks passed
<group>ALLCOMP_attributes</group>
<values>
<value>.false.</value>
<value COMP_WAV='ww3'>.true.</value>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alperaltuntas I just noticed that this is now on all the time if ww3 is used, that's not what we want is it?

Copy link
Member Author

@alperaltuntas alperaltuntas Feb 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, we'd like ww3 diagnostics to be enabled at all times. We don't expect any noticeable data or performance hit from this, since we only output surface diagnostics fields on a 3-degree grid.

@jedwards4b
Copy link
Collaborator

jedwards4b commented Feb 5, 2026 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants