8000 Documentation request and organization suggestion: MODIS sea ice observation converter · Issue #419 · NCAR/DART · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Documentation request and organization suggestion: MODIS sea ice observation converter #419

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.

Sign up for GitHub

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

Jump to bottom
Open
johnsonbk opened this issue Oct 27, 2022 · 1 comment
Open

Documentation request and organization suggestion: MODIS sea ice observation converter #419

johnsonbk opened this issue Oct 27, 2022 · 1 comment
Labels
obs_converters converting observations to DART format

Comments

@johnsonbk
Copy link
Collaborator

Use case

Users of various models such as CICE and FESOM may want to assimilate observations of sea ice extent from the the Moderate Resolution Imaging Spectroradiometer (MODIS).

DART has an observation converter for sea ice extent from MODIS but it is in the CICE directory.

The MODIS observation converter directory contains a converter for Land Product Subsets Leaf Area Index (LAI) and Fraction of Photosynthetically Active Radiation (FPAR):

ls DART/observations/obs_converters/MODIS
[ ... ]
MOD15A2_to_obs.f90
[ ... ]

While the MODIS sea ice extent converter is here:
DART/observations/obs_converters/cice/modis_ist_to_obs_netcdf.f90

Is your feature request related to a problem?

Observation converters are typically organized by the source of the observation rather than by the typical model that would assimilate the observation. The usual exceptions to this rule are when a given model computes its own forward operators and outputs them to a file, such as ROMS, and then the observation converter is used to generate an obs_seq file from the model-produced file.

Describe your preferred solution

It may be more navigable for a typical DART user if the MODIS sea ice extent obs_converter is moved from DART/observations/obs_converters/cice/ to DART/observations/obs_converters/MODIS/.

Describe any alternatives you have considered

Since we have many more CICE users than FESOM users, we could always just leave the MODIS obs_converter in DART/observations/obs_converters/cice/ and make a note in the MODIS documentation that the obs_converter for sea ice is in the cice directory.

In either case, there isn't any searchable documentation that describes that the observation converter for sea ice extent from MODIS actually exists, so at the very least, such documentation should be created (probably by me).
@braczka
Copy link
Contributor
braczka commented Nov 9, 2022

@johnsonbk I am OK with whatever you choose -- if you think it best to combine the MODIS sea ice obs converter within the DART/observations/obs_converters/MODIS folder along with the leaf area and fpar land converter, go for it. Like you said, it has become more standard to organize the obs converters by observation platform, rather than by model. Organizing the converters by model gives the impression that the observation converter is only functional within that model, which probably should not be the case. Regardless, as long as you provide documentation within the model page: /DART/models/cice/readme.rst, for example, where the common obs converters for cice are located, I think users will be able to navigate to it pretty easily.

@hkershaw-brown hkershaw-brown added the obs_converters converting observations to DART format label Nov 27, 2023
@hkershaw-brown hkershaw-brown mentioned this issue Apr 29, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
obs_converters converting observations to DART format
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@braczka @johnsonbk @hkershaw-brown
0