-
Notifications
You must be signed in to change notification settings - Fork 387
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
Fixed exact restart (temporarily) - classic and image drivers #507
Conversation
for both classic and image driver (This is a temporary solution to ensure exact restart)
The classic driver changes look good to me. I didn't look closely at the image driver changes. As an aside, I'm porting the classic driver changes to my fork of 4.2 and they appear to be functioning correctly there - this is a big help to me - thanks @yixinmao! |
@yixinmao - this looks good and can be merged, but first, can you add a bug fix note in the release notes: http://vic.readthedocs.io/en/develop/Development/ReleaseNotes/#bug-fixes. |
@jhamman I've edited the release notes. |
This PR assured exact restarts for classic and image driver under simple options (more thorough tests to be done). Specifically:
energy->LongUnderOut
andenergy->snow_flux
are now saved to state file. This is a temporary solution to solving the main points in issue Exact restart - classic driver #479 and assuring exact restart.