-
Notifications
You must be signed in to change notification settings - Fork 35
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
ctest 77% tests passed, 36 tests failed out of 159 (Debian GNU/Linux 10 ) #9
Comments
@Narayana-Rao Thanks for trying out isce3 and reporting this issue. I think that syntax is only valid in Python version 3.8, and I presume you're using an earlier version. We've already revised this part of the code in our development version to support Python 3.7. Maybe @gunter-jpl can suggest a workaround. |
rebase PR with the most recent develop
Yup, sorry, we had patched this issue internally but I neglected to sync the updates to the public repo. I just pushed the latest changes to develop. @Narayana-Rao, the issue should now be resolved as of 42af64e. Can you checkout the latest version of the code and give it another try? Thanks! |
Thank you @gmgunter and @bhawkins-jpl for the prompt response. I have tried with the updated version, and the count of failed tests is reduced to 6. The following is the list of failed tests (LastTest.log)
I think some version issues with GDAL. Except for
|
Hi @Narayana-Rao, thanks for pointing this out -- I wasn't aware of this issue. It looks like you're correct that the issue has to do with the GDAL version. ISCE3 relies on the After some discussion with @gshiroma about the best path forward, I think we will be dropping support for GDAL 2.x. It's not clear to me that we can reproduce the functionality that we need w/ older GDAL versions, and GDAL 3.0 has been available for ~3 years now. Could you please upgrade your GDAL version to 3.x in order to resolve this issue? Thanks again for the report. |
* account for DEM ref_height when computed EL bracket * Set ref_height to mean_height manually. * Actually enforce ref_height==mean_height in the DEMInterpolator class. * Revert "Set ref_height to mean_height manually." This reverts commit d739cb63c708dc46b62ce5ae6461757b59e4c45a. * Revert "account for DEM ref_height when computed EL bracket" This reverts commit 412fd2c8ec422adc516e942d7c3f221457b58be7. * I must not tell lies. * Add DEM coverage check * i can haz docstringz =^.^= * avoid loading large files multiple times * add unit test * Add data file used by new unit test. * require shapely, just not plotting * add API for PGE to calcuate TF area overlap * clarify coordinate system * add missing import
I am trying to install ISCE3 on a Debian GNU/Linux 10 machine. While checking the installation with ctest, I have encountered the following errors. Any help would be appreciated (LastTest.log ).
Most these failed tests has a common block of code
The text was updated successfully, but these errors were encountered: