Skip to content

Feature/cd snicar snow#324

Open
chengdang wants to merge 9 commits into
feature/btj_REL-3.2.0from
feature/cd_snicar_snow
Open

Feature/cd snicar snow#324
chengdang wants to merge 9 commits into
feature/btj_REL-3.2.0from
feature/cd_snicar_snow

Conversation

@chengdang

@chengdang chengdang commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Description

This PR adds a new VIS/NIR snow reflectance LUT to CRTM: SNICAR.VISsnow.EmisCoeff.nc

The following modules are modified/developed:

  • NetCDF LUT loading modules (binary format not supported)
  • Surface optics interpolation based on this new LUT
  • Forward calculation with this new LUT (TL/AD are under dev and will be added later in a separate PR)

To test this PR, please add this LUT in fix folder as:
EmisCoeff/VIS_Snow/SNICAR/netCDF/SNICAR.VISsnow.EmisCoeff.nc
https://drive.google.com/file/d/1o7LwBqykBoOQxOEvLwKFExFEluFvqYSg/view?usp=drive_link

Issue(s) addressed

#316 #323 #313

Dependencies

List the other PRs that this PR is dependent on:
None

Impact

None with the default LUT

Checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have run the unit tests before creating the PR

Note

To use this LUT in JEDI, the following snow properties are needed as part of surface geovals:

    sfc%Snow_Grain_Size  
    sfc%Snow_Density
    sfc%Snow_Depth

Existing surface property sfc%Snow_Type is not needed.

@chengdang chengdang self-assigned this Jun 23, 2026
@chengdang chengdang added the CRTM CRTM label Jun 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CRTM CRTM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant