Update inline post for multiple domains

[DusanJovic-NOAA]

Description

Provide a detailed description of what this PR does.
This PR updates inline post to support multiple domains (nests).

What bug does it fix, or what feature does it add? Multiple domains in inline post
Is a change of answers expected from this PR? Yes.

Issue(s) addressed

Link the issues to be closed with this PR, whether in this repository, or in another repository.
(Remember, issues should always be created before starting work on a PR branch!)

• fixes Enable inline post on multiple atm output grids in HAFS application ufs-community/ufs-weather-model#1306

Testing

How were these changes tested? Regression test on Hera
What compilers / HPCs was it tested with? Hera/Intel
Are the changes covered by regression tests? Yes.
Have the ufs-weather-model regression test been run? On what platform? Yes. Hera.

• Will the code updates change regression test baseline? Yes. two HAFS tests will run inline post.

Dependencies

If testing this branch requires non-default branches in other repositories, list them.
Those branches should have matching names (ideally)

Do PRs in upstream repositories need to be merged first? Yes.
If so add the "waiting for other repos" label and list the upstream PRs

• waiting on Support multiple domains in UFS inline post UPP#538

[jkbk2004]

RT test runs all passed for the upstream PR 1362. Can we have final approvals and start merging this pr?

[SMoorthi-emc]

Why not set default to YES for inline post? I hope most people want to use inline post. Moorthi

…

On Thu, Sep 1, 2022 at 9:54 AM Jun Wang ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In CMakeLists.txt <#570 (comment)>: > @@ -25,10 +25,8 @@ add_subdirectory(atmos_cubed_sphere) if(INLINE_POST) set(BUILD_POSTEXEC OFF) add_subdirectory(upp) - set(POST_SRC io/inline_post.F90 io/post_nems_routines.F90 io/post_fv3.F90) -else() Does this mean inline_post will be always on? — Reply to this email directly, view it on GitHub <#570 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ALLVRYTLCYRE6QD45ZCEQV3V4CYPNANCNFSM55K355KA> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

-- Dr. Shrinivas Moorthi Research Meteorologist Modeling and Data Assimilation Branch Environmental Modeling Center / National Centers for Environmental Prediction 5830 University Research Court - (W/NP23), College Park MD 20740 USA Tel: (301)683-3718 e-mail: ***@***.*** Phone: (301) 683-3718 Fax: (301) 683-3718

[DusanJovic-NOAA]

Why not set default to YES for inline post? I hope most people want to use inline post. Moorthi

I agree, it should be ON by default. I think initially, when inline post was added, it was running only on few platforms, so the default was made to be OFF, and it was turned ON on those platforms that supported it. At this moment I believe inline post runs on most platforms.

[jkbk2004]

@BrianCurtis-NOAA @DeniseWorthen we need one more approval from cm group.