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

Fates fuel refactor #2782

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

adrifoster
Copy link
Collaborator

Description of changes

Companion to FATES PR 1247

Specific notes

Just updates the location of a constant.

Are answers expected to change (and if so in what way)?

FATES PR does change answers for FATES cases, but changes are round-off due to some order of operations changes.

Any User Interface Changes (namelist or namelist defaults changes)? No

Does this create a need to change or add documentation? Did you do so? No

Testing performed, if any:

Running aux_clm and fates suites

@adrifoster
Copy link
Collaborator Author

all aux_clm and fates test suite tests PASS (or expected fail) except for BASELINE DIFFs as expected:

aux_clm: /glade/derecho/scratch/afoster/tests_0924-165752de

fates: /glade/derecho/scratch/afoster/tests_0924-151742de

confirming that DIFFs are round off as expected. Will report back

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In progress - master/b4b-dev
Development

Successfully merging this pull request may close these issues.

1 participant