Skip to content

Commit

Permalink
Merge branch 'hotfix-v6.1'
Browse files Browse the repository at this point in the history
This merge introduces v6.1 of MPAS.
Changes only occur in the sea ice model.

* hotfix-v6.1:
  Update README and LICENSE
  Increment version number to 6.1
  Removed reference to restartability_bgc in testing system.
  Added restart testing capability to restartability.py
  Added BGC testing and corrected 0 dimension
  Corrects iron uptake when carbon is active.
  Commented out two flags and updated bgc parameter defaults.
  • Loading branch information
matthewhoffman committed May 11, 2018
2 parents ece5f71 + 7c0e7e6 commit c970b61
Show file tree
Hide file tree
Showing 13 changed files with 80 additions and 63 deletions.
2 changes: 1 addition & 1 deletion LICENSE
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
Copyright (c) 2013, Los Alamos National Security, LLC (LANS) (Ocean: LA-CC-13-047;
Copyright (c) 2013-2018, Los Alamos National Security, LLC (LANS) (Ocean: LA-CC-13-047;
Land Ice: LA-CC-13-117) and the University Corporation for Atmospheric Research (UCAR).

All rights reserved.
Expand Down
26 changes: 15 additions & 11 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
MPAS-v6.0
MPAS-v6.1
====

The Model for Prediction Across Scales (MPAS) is a collaborative project for
Expand Down Expand Up @@ -28,25 +28,29 @@ core's user's guide, which can be found at the following web sites:

[MPAS-Atmosphere](http://mpas-dev.github.io/atmosphere/atmosphere_download.html)

[MPAS-Land Ice](http://mpas-dev.github.io/land_ice/download.html)
[MPAS-Albany Land Ice](http://mpas-dev.github.io/land_ice/download.html)

[MPAS-Ocean](http://mpas-dev.github.io/ocean/releases.html)

[MPAS-Seaice](http://mpas-dev.github.io/sea_ice/releases.html)


Code Layout
----------

Within the MPAS repository, code is laid out as follows. Sub-directories are
only described below the src directory.

MPAS
└── src
   ├── registry -- Code for building Registry.xml parser (Shared)
   ├── driver -- Main driver for MPAS in stand-alone mode (Shared)
   ├── external -- External software for MPAS (Shared)
   ├── framework -- MPAS Framework (Includes DDT Descriptions, and shared routines. Shared)
   ├── operators -- MPAS Opeartors (Includes Operators for MPAS meshes. Shared)
   ├── inc -- Empty directory for include files that Registry generates (Shared)
   └── core_* -- Individual model cores.
MPAS-Model
├── src
│   ├── registry -- Code for building Registry.xml parser (Shared)
│   ├── driver -- Main driver for MPAS in stand-alone mode (Shared)
│   ├── external -- External software for MPAS (Shared)
│   ├── framework -- MPAS Framework (Includes DDT Descriptions, and shared routines. Shared)
│   ├── operators -- MPAS Opeartors (Includes Operators for MPAS meshes. Shared)
│   ├── inc -- Empty directory for include files that Registry generates (Shared)
│   └── core_* -- Individual model cores.
└────── testing_and_setup -- tools for setting up configurations and tests cases (Shared)

Model cores are typically developed independently. For information about
building and running a particular core, please refer to that core's user's
Expand Down
2 changes: 1 addition & 1 deletion src/core_atmosphere/Registry.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0"?>
<registry model="mpas" core="atmosphere" core_abbrev="atm" version="6.0">
<registry model="mpas" core="atmosphere" core_abbrev="atm" version="6.1">

<!-- **************************************************************************************** -->
<!-- ************************************** Dimensions ************************************** -->
Expand Down
2 changes: 1 addition & 1 deletion src/core_init_atmosphere/Registry.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0"?>
<registry model="mpas" core="init_atmosphere" core_abbrev="init_atm" version="6.0">
<registry model="mpas" core="init_atmosphere" core_abbrev="init_atm" version="6.1">

<!-- **************************************************************************************** -->
<!-- ************************************** Dimensions ************************************** -->
Expand Down
2 changes: 1 addition & 1 deletion src/core_landice/Registry.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0"?>
<registry model="mpas" core="landice" core_abbrev="li" version="6.0">
<registry model="mpas" core="landice" core_abbrev="li" version="6.1">


<!-- ======================================================================= -->
Expand Down
2 changes: 1 addition & 1 deletion src/core_ocean/Registry.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0"?>
<registry model="mpas" core="ocean" core_abbrev="ocn" version="6.0">
<registry model="mpas" core="ocean" core_abbrev="ocn" version="6.1">

<dims>
<dim name="nCells" units="unitless"
Expand Down
76 changes: 38 additions & 38 deletions src/core_seaice/Registry.xml

Large diffs are not rendered by default.

7 changes: 4 additions & 3 deletions src/core_seaice/column/ice_algae.F90
Original file line number Diff line number Diff line change
Expand Up @@ -1278,7 +1278,7 @@ subroutine z_biogeochemistry (n_cat, dt, &
call add_warning(warning)
write(warning,*)'Category,mm:',n_cat,mm
call add_warning(warning)
l_stop = .true.
! l_stop = .true.
stop_label = 'zbgc FCT tracer solution failed'
endif
if (l_stop) return
Expand Down Expand Up @@ -1351,7 +1351,7 @@ subroutine z_biogeochemistry (n_cat, dt, &
call add_warning(warning)
write(warning, *) react(k,m),iphin_N(k),biomat_brine(k,m)
call add_warning(warning)
l_stop = .true.
! l_stop = .true.
stop_label = 'very large bgc value'
elseif (bio_tmp < c0) then
write(warning, *) 'negative bgc'
Expand Down Expand Up @@ -2036,7 +2036,8 @@ subroutine algal_dyn (dt, &
enddo
endif
endif !Docin(1) > c0
elseif (tr_bgc_Fe) then
endif
if (tr_bgc_Fe) then
do n = 1,n_fed
Fed_r(n) = Fed_r(n) + rFed(n)*Fed_tot_r ! scavenging + uptake
enddo
Expand Down
2 changes: 1 addition & 1 deletion src/core_sw/Registry.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0"?>
<registry model="mpas" core="sw" core_abbrev="sw" version="6.0">
<registry model="mpas" core="sw" core_abbrev="sw" version="6.1">
<dims>
<dim name="nCells"/>
<dim name="nEdges"/>
Expand Down
2 changes: 1 addition & 1 deletion src/core_test/Registry.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0"?>
<registry model="mpas" core="test" core_abbrev="test" version="6.0">
<registry model="mpas" core="test" core_abbrev="test" version="6.1">
<dims>
<dim name="nCells"/>
<dim name="nEdges"/>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@
config_use_vertical_tracers = true
config_use_skeletal_biochemistry = false
config_use_nitrate = true
config_use_carbon = true
config_use_carbon = false
config_use_chlorophyll = false
config_use_ammonium = true
config_use_silicate = true
Expand Down
14 changes: 12 additions & 2 deletions testing_and_setup/seaice/testing/tests/restartability.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,8 +59,18 @@ def restartability(mpasDevelopmentDir, domainsDir, domain, configuration, option
# restart
nProcs = 32

nmlChanges = {"seaice_model": {"config_start_time":"file"},
"restart": {"config_do_restart":True}}
bgcRestart = False
if ("bgc" in options.keys() and options["bgc"] == "True"):
bgcRestart = True

if (not bgcRestart):
nmlChanges = {"seaice_model": {"config_start_time":"file"},
"restart": {"config_do_restart":True}}
else:
nmlChanges = {"seaice_model": {"config_start_time":"file"},
"restart": {"config_do_restart":True,
"config_do_restart_bgc":True,
"config_do_restart_hbrine":True}}
if (check):
nmlChanges["unit_test"] = {"config_testing_system_test":True}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,9 @@
<domain name="domain_QU240km">
<test name="regression"/>
<test name="parallelism"/>
<test name="restartability"/>
<test name="restartability">
<option name="bgc" value="True"/>
</test>
</domain>
</configuration>
</testsuite>

0 comments on commit c970b61

Please sign in to comment.