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

Iono adding burst jumps #142

Merged
merged 11 commits into from
Jul 13, 2023
Merged

Iono adding burst jumps #142

merged 11 commits into from
Jul 13, 2023

Conversation

mgovorcin
Copy link
Contributor

@mgovorcin mgovorcin commented Jun 27, 2023

Ionosphere - estimating burst ramps due to high iono content (scalloping effect). This introduces mis-coregistration between bursts. Estimation is based on long-wavelength ionosphere correction estimate.

This PR turns on calculation of burst ramps by default, introducing additional iono processing steps: ionosphere_shift, ion2grd, esd that add 10 min of processing time for the fixed frame_id.

Ionosphere calculation outputs:

  1. dispersive long_wavelength ionosphere - obtained as 2order surface fitting to raw iono with addition of filtered raw_iono - surface residuals with adaptive gaussian. > merged/topophase.ion.geo

  2. azimuth shifts per bursts caused by ionosphere > merged/topophase.ion.az_shift.geo

cmd:
isce2_topsapp --secondary-scenes S1A_IW_SLC__1SDV_20220131T222803_20220131T222830_041711_04F690_8F5F S1A_IW_SLC__1SDV_20220131T222828_20220131T222855_041711_04F690_28D7 --reference-scenes S1A_IW_SLC__1SDV_20220212T222803_20220212T222830_041886_04FCA3_2B3E S1A_IW_SLC__1SDV_20220212T222828_20220212T222855_041886_04FCA3_A3E2 --frame-id 25502 --estimate-ionosphere-delay True

GUNW to inspect: https://gunw-development-testing.s3.us-west-2.amazonaws.com/iono_wmask_testing/S1-GUNW-A-R-164-tops-20220212_20220131-222829-00071W_00047N-PP-3d6c-v3_0_0/S1-GUNW-A-R-164-tops-20220212_20220131-222829-00071W_00047N-PP-3d6c-v3_0_0.nc
image

Result:
Long-wavelength iono : downsampled to resolution of 1 km
Burst ramps: 90m resution

image

Added metadata on ionosphere processing:

Long-wavelength layer:
image

Burst ramps layer:
image

@cmarshak
Copy link
Collaborator

cmarshak commented Jun 27, 2023

For packaging, take a look at #122 for how to add the layer to the /science/grid/data group.

Specifically:

https://github.com/ACCESS-Cloud-Based-InSAR/DockerizedTopsApp/blob/custom-event/isce2_topsapp/packaging_utils/additional_layers.json

and the updates to __main__.py (see here). We can talk through this.

You will want to make sure the output merged/topophase.ion.geo and merged/topophase.ion.az_shift.geo are the same affine transform as those other *.geo layers. Pretty sure they will be.

@cmarshak
Copy link
Collaborator

Awesome work!

@mgovorcin
Copy link
Contributor Author

mgovorcin commented Jul 12, 2023

For packaging, take a look at #122 for how to add the layer to the /science/grid/data group.

Specifically:

https://github.com/ACCESS-Cloud-Based-InSAR/DockerizedTopsApp/blob/custom-event/isce2_topsapp/packaging_utils/additional_layers.json

and the updates to __main__.py (see here). We can talk through this.

You will want to make sure the output merged/topophase.ion.geo and merged/topophase.ion.az_shift.geo are the same affine transform as those other *.geo layers. Pretty sure they will be.

added packaging :) @cmarshak and @dbekaert I believe this should be it what we discussed yesterday

@@ -6,6 +6,15 @@
"long_name": "ionospherePhaseCorrection",
"description": "Estimated ionosphere phase correction"
}
}
},
"ionosphereBurstRamps": {"dst_group": "/science/grids/corrections/derived/ionosphereBurstRamps",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

added to separate dst_group due to different resolution between ionosphere (downsampled) and ionosphereBurstRamps (full-res)

@mgovorcin mgovorcin requested a review from cmarshak July 12, 2023 19:04
cmarshak
cmarshak previously approved these changes Jul 12, 2023
Copy link
Collaborator

@cmarshak cmarshak left a comment

Choose a reason for hiding this comment

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

Thank you, Marin!

@cmarshak
Copy link
Collaborator

@mgovorcin - merge dev branch and push back. Hopefully that solves the pytest issues.

@cmarshak
Copy link
Collaborator

@dbekaert - I am going to merge this once this passes.

After that, we can do some tests over Chile, Arabia, and possibly northern parts of CONUS.

That way we can make sure this new methodology passes and look roughly at the stitching.

Hopefully that sounds good.

@cmarshak cmarshak merged commit 354991e into dev Jul 13, 2023
@cmarshak cmarshak deleted the iono_adding_burst_jumps branch July 13, 2023 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants