Integrate LOS vector in CSLC-S1 static layers product

opera-adt / COMPASS

COregistered Multi-temPorAl Sar Slc

Apache License 2.0

38 stars 17 forks source link

Integrate LOS vector in CSLC-S1 static layers product #181

Closed vbrancat closed 11 months ago

vbrancat commented 12 months ago

This PR integrates the LOS vector in the CSLC-S1 static layers product. Specifically, the PR removes the heading_angle layer and adds east_los_vector and north_los_vector.

TO DOs:

[x] 1. Merge ISCE3 PR computing LOS vectors in Topo
[x] 2. Test if the correct LOS vectors are returned for a couple of test bursts
[x] 3. Remove incidence_angle and heading
[x] 4. Check generated static layers product
[ ] 5. Modify static layers product specifications

vbrancat commented 12 months ago

No sure why the measure of the runtime is off when I ran the static layers:

journal: elapsed time (GEO-IN) [s]: 6.477
journal: s1_geocode_metadata burst successfully ran in 0:00:46 (hr:min:sec)
journal: s1_static_layers burst successfully ran in 19066 days, 15:54:45 (hr:min:sec)

Any idea how to fix this @LiangJYu ?

LiangJYu commented 12 months ago

No sure why the measure of the runtime is off when I ran the static layers:
journal: elapsed time (GEO-IN) [s]: 6.477
journal: s1_geocode_metadata burst successfully ran in 0:00:46 (hr:min:sec)
journal: s1_static_layers burst successfully ran in 19066 days, 15:54:45 (hr:min:sec)
Any idea how to fix this @LiangJYu ?

This line should look like this one.

vbrancat commented 12 months ago

@seongsujeong can you please review this PR? Thanks

LiangJYu commented 12 months ago

No sure why the measure of the runtime is off when I ran the static layers:
journal: elapsed time (GEO-IN) [s]: 6.477
journal: s1_geocode_metadata burst successfully ran in 0:00:46 (hr:min:sec)
journal: s1_static_layers burst successfully ran in 19066 days, 15:54:45 (hr:min:sec)
Any idea how to fix this @LiangJYu ?
This line should look like this one.

The 2 following places need time.time() changed to time.perf_counter() s1_geo2rdr.py s1_resample.py

LiangJYu commented 11 months ago

Process timing counter fixed and @seongsujeong comments addressed.