Skip to content

Conversation

@fvitt
Copy link
Collaborator

@fvitt fvitt commented Nov 17, 2020

Do not set use_hetfrz_classnuc if chemistry does not include MAM species.

Added aux_cam test which uses aquaplanet cam6 physics and -chem none.

closes #261

@fvitt fvitt added the BFB bit for bit tag label Nov 17, 2020
@fvitt fvitt added this to the CESM2.3 milestone Nov 17, 2020
@fvitt fvitt self-assigned this Nov 17, 2020
Copy link
Collaborator

@gold2718 gold2718 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks okay to me (assuming you add a ChangeLog entry), just one question.

Comment on lines 448 to 455
<test compset="QPC6" grid="f10_f10_mg37" name="SMS_D_Ln9" testmods="cam/outfrq3s_ba">
<machines>
<machine name="izumi" compiler="nag" category="aux_cam"/>
</machines>
<options>
<option name="wallclock">00:30:00</option>
</options>
</test>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this test here instead of after the QPC5 version of this test (see line 677)?
Also, why does outfrq3s_ba mean no chemistry?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"ba" --> bulk aerosols which is what you get for '-chem none'.

@cacraigucar cacraigucar requested a review from nusbaume November 19, 2020 20:47
@cacraigucar cacraigucar changed the title Enable "-chem none" configure option with cam6 physics cam6_3_005: Enable "-chem none" configure option with cam6 physics Nov 24, 2020
@fvitt fvitt merged commit 065d247 into ESCOMP:cam_development Nov 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BFB bit for bit tag

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants