Skip to content

Conversation

@danielhollas
Copy link
Contributor

@danielhollas danielhollas commented Dec 1, 2023

TODO:

  • Fix tests
  • Make the restart more robust, print helpful error for old versions of restart files

@danielhollas danielhollas changed the title WIP: Add eshift to restart file to preserve continuity Add eshift to restart file to preserve continuity Dec 5, 2023
@danielhollas danielhollas marked this pull request as ready for review December 16, 2023 20:50
@danielhollas danielhollas added this to the Version 1.2 milestone Nov 27, 2024
This was referenced Oct 7, 2025
@codecov
Copy link

codecov bot commented Oct 8, 2025

Codecov Report

❌ Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.04%. Comparing base (85cf84f) to head (577e234).

Files with missing lines Patch % Lines
src/sh_integ.F90 66.66% 1 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #168      +/-   ##
==========================================
- Coverage   84.06%   84.04%   -0.03%     
==========================================
  Files          47       47              
  Lines        6602     6606       +4     
  Branches      764      765       +1     
==========================================
+ Hits         5550     5552       +2     
- Misses        853      854       +1     
- Partials      199      200       +1     
Flag Coverage Δ
unittests 18.39% <0.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/surfacehop.F90 89.15% <100.00%> (ø)
src/sh_integ.F90 91.05% <66.66%> (-0.65%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants