ASFHyP3 / hyp3-autorift

A HyP3 plugin for feature tracking processing with AutoRIFT-ISCE
BSD 3-Clause "New" or "Revised" License
7 stars 2 forks source link

Landsat 7 + 8 pairs may not be handled well #201

Closed jhkennedy closed 1 year ago

jhkennedy commented 1 year ago

Since we only look at the reference scene to determine the platform, we may have some issues with the secondary scene:

Fortunately, in our current campaign pair list, 4/5 pairs are only crossed with 4/5 pairs, and 9 pairs are only crossed with 8, so we should only see issues around 7+8 pairs.

>>> import geopandas as gpd
>>> df = gpd.read_parquet('l45789.parquet')
>>> df.groupby(['ref_mission', 'sec_mission']).reference.count()
ref_mission  sec_mission
L4           L4                1058
             L5                3203
L5           L4                2988
             L5              767024
L7           L7             2003704
             L8              474339
L8           L7              416539
             L8             4103107
             L9              367178
L9           L8              112403
             L9              109936

Fix: AutoRIFT pre-processing defaults to the strictest filtering, so we should allow L8 scenes to be wallis filled if either pair is L7. And we should check both ref, sec platform when deciding to run the pre-processing filters or not in process.py