Skip to content

Add individual correction metadata to CSLC-S1 #156

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

Merged
vbrancat merged 5 commits into from
May 26, 2023
Merged

Add individual correction metadata to CSLC-S1 #156

Changes from 3 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
f562933
Allocate flattening flags to true
May 25, 2023
40bf7ae
Avoid mixing camel and snake case
May 25, 2023
6d54c20
Add individual correction flag to indicate which correction have been…
May 25, 2023
1224889
Discriminate between wet and dry tropo corrections
May 26, 2023
ec4e0dd
Add azimuth SET flag placeholder
May 26, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 20 additions & 6 deletions src/compass/utils/h5_helpers.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -567,13 +567,27 @@ def metadata_to_h5group(parent_group, burst, cfg):
add_dataset_and_attrs(burst_meta_group, meta_item)

# Add parameters group in processing information
tropo_corr = True if cfg.weather_model_file is not None else False
tec_corr = True if cfg.tec_file is not None else False
par_meta_items = [
Meta('ellipsoidalFlatteningApplied', cfg.geocoding_params.flatten,
"If True, CSLC-S1 phase has been flatten with respect to a zero height ellipsoid",
{'units':'unitless'}),
Meta('topographicFlatteningApplied', cfg.geocoding_params.flatten,
"If True, CSLC-S1 phase has been flatten with respect to topographic height using a DEM",
{'units': 'unitless'}),
Meta('ellipsoidal_flattening_applied', bool(cfg.geocoding_params.flatten),
"If True, CSLC-S1 phase has been flatten with respect to a zero height ellipsoid"),
Meta('topographic_flattening_applied', bool(cfg.geocoding_params.flatten),
"If True, CSLC-S1 phase has been flatten with respect to topographic height using a DEM"),
Meta('bistatic_delay_applied', bool(cfg.lut_params.enabled),
"If True, bistatic delay correction has been applied"),
Meta('azimuth_fm_rate_applied', bool(cfg.lut_params.enabled),
"If True, azimuth FM-rate mismatch correction has been applied"),
Meta('geometry_doppler_applied', bool(cfg.lut_params.enabled),
"If True, geometry steering doppler correction has been applied"),
Meta('solid_earth_tides_applied', bool(cfg.lut_params.enabled),
"If True, solid Earth tides correction has been applied"),
Meta('static_troposphere_applied', bool(cfg.lut_params.enabled),
"If True, troposphere correction based on a static model has been applied"),
Comment on lines +590 to +591
Copy link
Copy Markdown
Contributor

@seongsujeong seongsujeong May 26, 2023

Choose a reason for hiding this comment

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

What is going to happen for static troposphere correction if cfg.lut_params.enabled is True, and we apply weather-model based troposphere correction? I think this field has to be False in that case?

Copy link
Copy Markdown
Author

@vbrancat vbrancat May 26, 2023

Choose a reason for hiding this comment

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

My hope is that when we include the individual enabling/disabling of the LUTs, each of these flags will be replaced by its own individual flag. So later on, we should have the ability to set to False (or True) each of these corrections

Meta('ionosphere_tec_applied', tec_corr,
"If True, ionosphere correction based on TEC data has been applied"),
Meta('troposphere_weather_model_applied', tropo_corr,
"If True, troposphere correction based on weather model has been applied")
]
par_meta_group = processing_group.require_group('parameters')
for meta_item in par_meta_items:
Expand Down