Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug fixes to enable aerosols & dust in multi-layer snow with Snicar-ad #818

Merged

Conversation

njeffery
Copy link
Contributor

This PR allows for radiative transfer calculations of aerosols and dust in seaice. The capability will currently remain off as default.

  1. Corrects the snow surface scattering layer and bgc snow grid for aerosols with multiple snow layers
  2. adds a minimum snow thickness for direct deposition of aerosols in snow.
  3. adds a length scale for penetration of surface atm flux into the snow interior for thin snow layers.
  4. Alters the aerosol testing suite configuration to test multiple snow layers and snicar-ad
  5. defines diagnostic verticalAerosolSnowCell
  6. corrects unit definitions for aerosol fields in Registry

 -Snow surface layer thickness now properly defined  for multiple snow
 layers.
-Added a length scale and code for spreading  surface atmospheric fluxes into
 the snow interior.  High concentrations, for very thin snow layers and
 thick sea ice,  will cause the snicar-ad code
 to crash.
 -Added a minimum snow thickness condition for atm-aerosol fluxes into
 the snow.  BC and dust now enter the sea ice directly for very thin
 snow thicknesses.
-corrected aerosol  units mg/m3 to kg/m3
-diagnostic verticalSnowAerosolCell now defined
Enabled multiple snow layers and snicar-ad in aerosol test
cleaned up ice_shortwave error messages and typpos
@njeffery
Copy link
Contributor Author

This code is BFB in the WC configuration. All five testsuites performed and passed except the standard_bgc and the aerosol_shortwave regression tests. This is expected because the PR has changes to the aerosol transport and snow grid are aerosols are enabled in the standard_bgc test.

@akturner akturner self-assigned this Feb 24, 2021
@proteanplanet
Copy link

@njeffery "This code is BFB in the WC configuration": Does this mean you've run a B-case with this? Was it BFB for a full annual cycle?

@njeffery
Copy link
Contributor Author

njeffery commented Mar 8, 2021 via email

Copy link

@proteanplanet proteanplanet left a comment

Choose a reason for hiding this comment

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

Approved by visual inspection and evidence of testing.

@akturner akturner merged commit 382b432 into MPAS-Dev:seaice/develop Mar 11, 2021
jonbob added a commit to E3SM-Project/E3SM that referenced this pull request Mar 15, 2021
Update mpas-source: Allows for sea ice radiative transfer calculations

Brings in a new mpas-source submodule with changes ony to the seaice
core. It allows for sea ice radiative transfer calculations of aerosols
and dust.

Merges MPAS-Dev/MPAS-Model#818

[Non-BFB] for aerosol/BGC sims
jonbob added a commit to E3SM-Project/E3SM that referenced this pull request Mar 16, 2021
Update mpas-source: Allows for sea ice radiative transfer calculations

Brings in a new mpas-source submodule with changes ony to the seaice
core. It allows for sea ice radiative transfer calculations of aerosols
and dust.

Merges MPAS-Dev/MPAS-Model#818

[Non-BFB] for aerosol/BGC sims
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants