Skip to content

Add code to download SLCs from JPL - #47

Open
julober wants to merge 9 commits into
SnowEx:mainfrom
julober:main
Open

Add code to download SLCs from JPL #47
julober wants to merge 9 commits into
SnowEx:mainfrom
julober:main

Conversation

@julober

@julober julober commented Mar 5, 2026

Copy link
Copy Markdown
Collaborator

I wrote a couple of functions to generate the UAVSAR SLC filenames (from asf search results) and download them from the JPL site, which I thought might be useful to add to this repository. There are a couple of things that I couldn't find in the ASF metadata (what I call tag and the release folder). For the release folder, the code just brute force searches for a valid link, and for the tag, I let the user determine what tags to look for. I tested them in a notebook called slc_download_test.ipynb. Let me know if you have any feedback!

@julober

julober commented Mar 8, 2026

Copy link
Copy Markdown
Collaborator Author

After testing the code for myself, I determined that I needed to search both tags (BC/BU) in a similar way as the release folder, so I made that change.

@ZachHoppinen
ZachHoppinen self-requested a review September 3, 2026 23:42

@ZachHoppinen ZachHoppinen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This all looks good to me, feel free to merge.

For versioning, I agree it is not the prettiest, but I do not think there is a cleaner option given that ASF's versioning and the SLC stack versioning on the UAVSAR site are independent. The one thing that might help later is resolving the release folder and tag inside get_uavsar_slcs instead of at download time. It already makes a network call to ASF, so one extra probe per flight line is cheap, and it would let you return real URLs rather than the [BC/BU] placeholder and surface a bad version where the user typed it.

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