Skip to content

Commit

Permalink
fix typo and white space.
Browse files Browse the repository at this point in the history
git-svn-id: https://svn-dares-dart.cgd.ucar.edu/DART/releases/Manhattan@13138 dfa8782c-da17-4c45-ba5c-5625b50a00d6

Former-commit-id: 17bdf80651381b94bc81c619a97c90f811bbb3e0
  • Loading branch information
nancycollins committed Apr 30, 2019
1 parent 64b29b6 commit 8aef552
Showing 1 changed file with 13 additions and 11 deletions.
24 changes: 13 additions & 11 deletions CHANGELOG
Original file line number Diff line number Diff line change
Expand Up @@ -775,15 +775,17 @@ and assim_tools_mod.pf.f90 in sync with assim_tools_mod.f90.
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

- Important performance fix if model does vertical conversion for localization.
Results were not wrong but performance was poor if 'distribute_mean = .true.'
was selected in the &assim_tools_nml namelist.
Now distributing the mean runs in close to the non-distributed time and uses
much less memory for large models. This only impacts models which do a vertical
conversion of either the observation or state vertical coordinate for localization
AND which set &assim_tools_nml :: distribute_mean = .true. to use less memory.
When using a distributed mean "convert_all_obs_verticals_first = .true." should
be set. If your observations will impact most of the model state, then
"convert_all_state_verticals_first = .true.' can also be set.
Results were not wrong but performance was poor if 'distribute_mean = .true.'
was selected in the &assim_tools_nml namelist.

Now distributing the mean runs in close to the non-distributed time and uses
much less memory for large models. This only impacts models which do a vertical
conversion of either the observation or state vertical coordinate for localization
AND which set &assim_tools_nml :: distribute_mean = .true. to use less memory.

When using a distributed mean "convert_all_obs_verticals_first = .true." should
be set. If your observations will impact most of the model state, then
"convert_all_state_verticals_first = .true.' can also be set.

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
+ Apr 30 2019 :: cam-fv refactor, posteriors optional, QC 8 $Revision$
Expand Down Expand Up @@ -825,7 +827,7 @@ be set. If your observations will impact most of the model state, then
- get_maxdist() now a required routine all location modules

- Default routines now create a time variable as time(time) to allow multiple
files to be concatinated along the unlimited dimension more easily. Also
files to be concatenated along the unlimited dimension more easily. Also
conforms to the netCDF convention for coordinate dimensions.

- obs_impact_tool handles a continuum of values, not just discrete 0 or 1.
Expand Down Expand Up @@ -864,7 +866,7 @@ be set. If your observations will impact most of the model state, then
- Better error and informational messages in various routines.

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
+ MMM DD YYYY :: summary of changes in Manhattan update $Revision$
+ MMM DD YYYY :: summary of changes in next Manhattan update $Revision$
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++


Expand Down

0 comments on commit 8aef552

Please sign in to comment.