Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added the phen_model_day timer to work with all restart conditions #685

Merged
merged 7 commits into from
Aug 26, 2022

Conversation

rgknox
Copy link
Contributor

@rgknox rgknox commented Aug 14, 2020

Description:

The phen_model_day timer was added to the site structure. This timer is initialized as the day of year on a cold-start, and is read in from the restart file. This replaces the hlm_model_day, which for some types of restarts, will not be continuous from the last simulation. (ie it may be reset) By using phen_model_day, we have consistency across restarts, and this absolute integer day will always be preserved.

Note: Users who restart the model via an finidat, or non CONTINUE_RUN simulation, AND, have a gap in the DAY OF YEAR from the end of one simulation to the start of the next, will likely still encounter phenology weirdness. However, it seems reasonable that they should expect weirdness if they decide to magically remove a portion of the year from the plants life-cycle.

Fixes #627
Fixes #894

Collaborators:

@rosiealice @adrifoster

Expectation of Answer Changes:

Yes, this will change answers, but only on non CONTINUE_RUN restart scenarios, and only on deciduous vegetation.

Checklist:

  • I have updated the in-code documentation .AND. (the technical note .OR. the wiki) accordingly.
  • I have read the CONTRIBUTING document.
  • FATES PASS/FAIL regression tests were run
  • If answers were expected to change, evaluation was performed and provided

Test Results:

TBD

CTSM (or) E3SM (specify which) test hash-tag:

CTSM (or) E3SM (specify which) baseline hash-tag:

FATES baseline hash-tag:

Test Output:

@rgknox
Copy link
Contributor Author

rgknox commented Aug 14, 2020

See test results from 2 simulations at bci, 10 years each, 4 pfts (our drought deciduous pfts), the base is master, the test is this branch.
PMD-test_plots.pdf

Running a hybrid test now, where I start a second simulation in the 5th out of 10th year. In a hybrid, the reference date should change, so this test should indicate if the our timers are preserved through the restart.

rgknox added 2 commits August 14, 2020 19:42
…model day counter to start at 0, and +1 at start of phenology routine, which is simpler for restarts.
@rgknox
Copy link
Contributor Author

rgknox commented Aug 17, 2020

hybrid test looks good. I ran two simulations, 1) a base simulation that was a 10 year cold-start, and a 5 year hybrid run that used the base simulations 5th year as an finidat (initial condition). Model output from the hybrid run and the base run are indistinguishable, included "days since leaf on and leaf off" diagnostics.

hybr-test2_plots.pdf

Copy link
Contributor

@rosiealice rosiealice left a comment

Choose a reason for hiding this comment

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

Sorry for taking so long to have the headspace to get to this! It all looks fine to me, and I think will prove useful for ongoing phenology updates...

@rgknox rgknox added the PR status: Not Ready The author is signaling that this PR is a work in progress and not ready for integration. label Aug 16, 2021
@rgknox rgknox removed the PR status: Not Ready The author is signaling that this PR is a work in progress and not ready for integration. label Aug 22, 2022
@rgknox
Copy link
Contributor Author

rgknox commented Aug 25, 2022

tested on cheyenne, "fates" test suite:
/glade/scratch/rgknox/tests_0825-130322ch
I believe tests are nominal (still checking), all tests have expected diffs with BASELINE, but the diffs are only in the variables that had their calculations changed:

 FATES_DAYSINCE_COLDLEAFOFF
 FATES_DAYSINCE_COLDLEAFON
 FATES_DAYSINCE_DROUGHTLEAFOFF
 FATES_DAYSINCE_DROUGHTLEAFON

Expected FAILS:
ERS_Lm12.1x1_brazil.I2000Clm50FatesCruRsGs.cheyenne_intel.clm-FatesFireLightningPopDens COMPARE_base_rest
ERS_D_Ld15.f45_f45_mg37.I2000Clm50FatesRs.cheyenne_intel.clm-FatesColdDefTreeDamage

@rgknox rgknox merged commit 08d429d into NGEET:master Aug 26, 2022
@rgknox rgknox deleted the phen-day-counter branch October 31, 2023 17:46
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.

Phenology flags get locked in when starting. anew case using. prior case's restart Phenology and restarting
2 participants