Loosen restriction on optional arguments in CCPP Framework

[dustinswales]

Commit Queue Requirements:

• Fill out all sections of this template.
• All sub component pull requests have been reviewed by their code managers.
• Run the full Intel+GNU RT suite (compared to current baselines) on either Hera/Derecho/Hercules
• Commit 'test_changes.list' from previous step

---

Description:

Commit Message:

* UFSWM - Loosen restriction on optional arguments in CCPP.

Priority:

• Normal.

Git Tracking

UFSWM:

• None

Sub component Pull Requests:

• FV3: Loosen restriction on optional arguments in CCPP. NOAA-EMC/ufsatm#917)
  • ccpp-physics: Loosen restriction on optional arguments. ccpp-physics#248)

UFSWM Blocking Dependencies:

• None

Changes

Regression Test Changes (Please commit test_changes.list):

• No Baseline Changes.
  (Waiting on full set of RTs on Hera, Will update this PR when tests finished)

Input data Changes:

• None.

Library Changes/Upgrades:

• No Updates

---

Testing Log:

• RDHPCS
  • Hera
  • Orion
  • Hercules
  • GaeaC5
  • GaeaC6
  • Derecho
• WCOSS2
  • Dogwood/Cactus
  • Acorn
• CI
• opnReqTest (complete task if unnecessary)

[grantfirl]

@dustinswales Do you have any RT logs? I'm guessing this isn't supposed to change baselines? Let me know if you'd like me to run RTs on this.

[dustinswales]

@grantfirl I'm addressing your issues in the physics and will rerun the RTs.
Thanks for syncing my branches!

[grantfirl]

@grantfirl I'm addressing your issues in the physics and will rerun the RTs. Thanks for syncing my branches!

I think that I fixed the inconsistency that I pointed out. I don't think anything else is required on the physics side. Just RTs at this point. You may want to double-check.

[grantfirl]

Note that the ccpp-framework PR changed to NCAR/ccpp-framework#638 and is already merged.

[dustinswales]

@grantfirl I'm addressing your issues in the physics and will rerun the RTs. Thanks for syncing my branches!

I think that I fixed the inconsistency that I pointed out. I don't think anything else is required on the physics side. Just RTs at this point. You may want to double-check.

@grantfirl I just saw that. Thanks!
I'll rerun the RTs now and update the logs.

[grantfirl]

@grantfirl I'm addressing your issues in the physics and will rerun the RTs. Thanks for syncing my branches!

I think that I fixed the inconsistency that I pointed out. I don't think anything else is required on the physics side. Just RTs at this point. You may want to double-check.

@grantfirl I just saw that. Thanks! I'll rerun the RTs now and update the logs.

@dustinswales Any update on these RTs?

[dustinswales]

@grantfirl I'm addressing your issues in the physics and will rerun the RTs. Thanks for syncing my branches!

I think that I fixed the inconsistency that I pointed out. I don't think anything else is required on the physics side. Just RTs at this point. You may want to double-check.

@grantfirl I just saw that. Thanks! I'll rerun the RTs now and update the logs.

@dustinswales Any update on these RTs?

@grantfirl Some tests are failing and I need to look into this. Sigh.

[jkbk2004]

@BrianCurtis-NOAA @FernandoAndrade-NOAA this pr is ready.

[jkbk2004]

we can start merging process.

[jkbk2004]

@dustinswales fv3 new hash is NOAA-EMC/ufsatm@de5d1c2

[jkbk2004]

Created an MOM6 code management issue: #2717