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

Fix to driver history #421

Merged
merged 1 commit into from
Dec 17, 2022
Merged

Fix to driver history #421

merged 1 commit into from
Dec 17, 2022

Conversation

dabail10
Copy link
Contributor

For detailed information about submitting Pull Requests (PRs) to the CICE-Consortium,
please refer to: https://github.com/CICE-Consortium/About-Us/wiki/Resource-Index#information-for-developers

PR checklist

There are some missing baselines for PGI here.
MISS cheyenne_pgi_smoke_col_1x1_diag1_run1year compare icepack.8f96707a90.221210-095008 missing-data
MISS cheyenne_pgi_smoke_col_1x1_debug_run1year compare icepack.8f96707a90.221210-095008 missing-data
MISS cheyenne_pgi_smoke_col_1x1_bgcispol_debug compare icepack.8f96707a90.221210-095008 missing-data
MISS cheyenne_pgi_smoke_col_1x1_bgcnice_debug compare icepack.8f96707a90.221210-095008 missing-data
MISS cheyenne_pgi_smoke_col_1x1_bgcsklnice_debug compare icepack.8f96707a90.221210-095008 missing-data
MISS cheyenne_pgi_smoke_col_1x1_debug_run1year_thermo1 compare icepack.8f96707a90.221210-095008 missing-data
MISS cheyenne_pgi_smoke_col_1x1_debug_run1year_swredist compare icepack.8f96707a90.221210-095008 missing-data
MISS cheyenne_pgi_smoke_col_1x1_debug_run1year_swccsm3 compare icepack.8f96707a90.221210-095008 missing-data
MISS cheyenne_pgi_smoke_col_1x1_alt01_debug_run1year compare icepack.8f96707a90.221210-095008 missing-data
MISS cheyenne_pgi_smoke_col_1x1_alt02_debug_run1year compare icepack.8f96707a90.221210-095008 missing-data
MISS cheyenne_pgi_smoke_col_1x1_alt03_debug_run1year compare icepack.8f96707a90.221210-095008 missing-data
MISS cheyenne_pgi_smoke_col_1x1_alt04_debug_run1year compare icepack.8f96707a90.221210-095008 missing-data
MISS cheyenne_pgi_smoke_col_1x1_debug_dt30min_leap_run1year compare icepack.8f96707a90.221210-095008 missing-data
MISS - comparison data is missing
13 of 495 tests MISSING data

https://github.com/CICE-Consortium/Test-Results/wiki/icepack_by_hash#8f96707a90132ca119d81ed84e5a62ca0ff3ed96

  • How much do the PR code changes differ from the unmodified code?
    • bit for bit
    • different at roundoff level
    • more substantial
  • Does this PR create or have dependencies on CICE or any other models?
    • Yes
    • No
  • Does this PR add any new test cases?
    • Yes
    • No
  • Is the documentation being updated? ("Documentation" includes information on the wiki or in the .rst files from doc/source/, which are used to create the online technical docs at https://readthedocs.org/projects/cice-consortium-cice/.)
    • Yes
    • No, does the documentation need to be updated at a later time?
      • Yes
      • No
  • Please provide any additional information or relevant details below:

Copy link
Contributor

@eclare108213 eclare108213 left a comment

Choose a reason for hiding this comment

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

Did you confirm that the history output for all variables except FSD remains the same as before?

start4(3) = 1
count4(3) = ntrcr
count4(3) = ncat
Copy link
Contributor

Choose a reason for hiding this comment

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

does this change only impact FSD output?

Copy link
Contributor

@cmbitz cmbitz Dec 14, 2022

Choose a reason for hiding this comment

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

It impacts all the variables in trcrn, which includes many other tracers. All of them are currently set to -9999 in the history file, even without the FSD. I did not verify that all other variables besides trcrn are unchanged. I'll do that later this afternoon.

@dabail10
Copy link
Contributor Author

The log files were bfb.

@eclare108213
Copy link
Contributor

The log files were bfb.

Yes, but this changes the history output. Does it look right?

@dabail10
Copy link
Contributor Author

I will run an io_suite.

@dabail10
Copy link
Contributor Author

I have confirmed that trcrn now has valid values where it had -9999 before.

Copy link
Contributor

@eclare108213 eclare108213 left a comment

Choose a reason for hiding this comment

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

Excellent. Thank you both, @cmbitz and @dabail10, for fixing this bug.

@apcraig
Copy link
Contributor

apcraig commented Dec 15, 2022

Do we need a similar fix in CICE?

@dabail10
Copy link
Contributor Author

I do not believe so. We do not dump the whole trcr/trcrn array in CICE like we do in Icepack.

@dabail10
Copy link
Contributor Author

This should be good to go.

@apcraig apcraig merged commit 3a4d02e into CICE-Consortium:main Dec 17, 2022
@dabail10 dabail10 deleted the fsdhist branch March 23, 2023 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants