Skip to content

fix get_param STOKES_MOST and WAVE_ENHANCED_USTAR in MOM.F90 #418

Merged
gustavo-marques merged 4 commits into
dev/ncarfrom
fix_unused_params
Apr 9, 2026
Merged

fix get_param STOKES_MOST and WAVE_ENHANCED_USTAR in MOM.F90 #418
gustavo-marques merged 4 commits into
dev/ncarfrom
fix_unused_params

Conversation

@alperaltuntas
Copy link
Copy Markdown
Member

@alperaltuntas alperaltuntas commented Apr 8, 2026

Enclose get_param calls for STOKES_MOST and WAVE_ENHANCED_USTAR with openParameterBlock/closeParameterBlock in MOM.F90. Previously these parameters weren't being read in. This PR changes answers.

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 37.29%. Comparing base (285fc12) to head (810ed7b).
⚠️ Report is 4 commits behind head on dev/ncar.

Additional details and impacted files
@@             Coverage Diff              @@
##           dev/ncar     #418      +/-   ##
============================================
- Coverage     37.29%   37.29%   -0.01%     
============================================
  Files           306      306              
  Lines         93998    94031      +33     
  Branches      18116    18117       +1     
============================================
+ Hits          35056    35066      +10     
- Misses        52307    52331      +24     
+ Partials       6635     6634       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gustavo-marques gustavo-marques merged commit 9f622bf into dev/ncar Apr 9, 2026
104 checks passed
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.

3 participants