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

remove initializations for multiple init calls #230

Merged
merged 1 commit into from
Nov 9, 2018

Conversation

eclare108213
Copy link
Contributor

Calls to init_tracer_indices without a complete argument list subsequent to the first call would reset the non-present indices to zero.

  • Developer(s): E. Hunke

  • Please suggest code Pull Request reviewers in the column at right.

  • Are the code changes bit for bit, different at roundoff level, or more substantial? BFB

  • To verify that this PR passes the initial QC tests, lease include the link to test results
    or paste in below the summary block from the bottom of the testing output.

  • Does this PR create or have dependencies on CICE or any other models? No

  • Is the documentation being updated with this PR? (Y/N) No
    If not, does the documentation need to be updated separately at a later time? (Y/N) No

Note: "Documentation" includes information on the wiki and .rst files in doc/source/,
which are used to create the online technical docs at https://readthedocs.org/projects/cice-consortium-cice/.

  • Other Relevant Details:

@apcraig apcraig merged commit 5d22a61 into CICE-Consortium:master Nov 9, 2018
@eclare108213 eclare108213 deleted the tracer_init_icepack branch November 21, 2018 18:36
lettie-roach pushed a commit to lettie-roach/Icepack that referenced this pull request Oct 18, 2022
* Record number for popgrid_nc (NetCDF) is always one

* HYCOM (NetCDF) I/O routines used by DMI

* Updated documentation for HYCOM forcing, precip units and restart dump in hours interval

* "hycom_atm" option changed to "hycom"

* option "HYCOM_atm" -> "hycom". Correct "near" -> "ncar"
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.

2 participants