[mpas-developers] proposed trunk commit

[Michael Duda]

Hi, Folks.

Here's another small change I'd like to make in the trunk, in
src/framework/mpas_framework.F (current on the left, proposed on the
right):


16c16
<    subroutine mpas_framework_init(dminfo, domain)
---
>    subroutine mpas_framework_init(dminfo, domain, mpi_comm)
21a22
>       integer, intent(in), optional :: mpi_comm
27c28
<       call mpas_dmpar_init(dminfo)
---
>       call mpas_dmpar_init(dminfo, mpi_comm)


Essentially, I'd like to be able to specify an optional MPI communicator
to be used by the dmpar module via the high-level mpas_framework_init()
call. The mpas_dmpar_init() routine already has an optional
communicator argument, so no changes are needed there.

This change should be (and is, in the case of the non-hydrostatic MPAS-A
in CAM) useful for MPAS cores running as components within other models.

Michael


On Wed, Aug 22, 2012 at 11:31:27AM -0600, Michael Duda wrote:
> Hi, All.
> 
> I'd like to remove a bit of deprecated code from the framework
> directory.  Specifically, our infrastructure is now general enough to
> handle any non-decomposed variable, so we no longer need special code to
> handle fields whose outermost dimension is nVertLevels. I'm proposing to
> remove all code that references nVertLevels (<10 lines total!) from
> mpas_io_input.F and mpas_block_creator.F; these changes have no effect
> on MPAS models.
> 
> If there are no objections, I'd like to commit these changes before the
> end of the week.
> 
> Thanks,
> Michael