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

ARIA-tools version used in container breaks with introduction of OpenTopography API key #10

Open
WalternativE opened this issue Aug 6, 2022 · 2 comments

Comments

@WalternativE
Copy link

First of all: thanks for your great work! I'm currently working through the Unavco 2021 InSAR training material and it would be downright impossible to do without this container. Reading the included container declaration and scripts is a valuable lesson on its own and I really want to express my utmost gratitude.

One problem I encountered was that the install_unavco_pkgs.sh script currently pins the ARIA-tools repo to release 1.1.2. This (as well as 1.1.3) do not include the fixes for the mandatory OpenTopography API key when downloading the DEM (e.g. when preparing time series for use in MintPy). I fixed it for me by removing the ARIA-tools directory in the .local folder and changing the git clone command in the install_unavco_pkgs.sh to default to the latest commit. I'm reluctant to write up a PR for this as I assume that you pinned the repository for a reason. Would you suggest to open an issue with the ARIA-tools repository to ask for a new release and pinning that instead?

@WalternativE
Copy link
Author

The newest version that includes this fix appears to break ariaTSsetup.py. This is connected to this issue aria-tools/ARIA-tools#307. I'll try to find a sensible workaround.

@WalternativE
Copy link
Author

I tried out using gdal versions 2.4.1, 2.4.2 and 2.4.3 with the newest updates in ARIA-tools. I'm blocked from trying out versions greater or equal than 2.5.x as isce2 isn't compatible with those. As a workaround I forked ARIA-tools and branched of the 1.1.2 release to just backport the OpenTopography API key changes. This isn't exactly elegant but it allowed me to run the ariaTSsetup_tutorial notebook using the unavco kernel.

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

No branches or pull requests

1 participant