-
Notifications
You must be signed in to change notification settings - Fork 2
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
Snaphu export fails for areas with NaN values (water, border pixels) #9
Comments
This comment has been minimized.
This comment has been minimized.
After further checking, what seems to be the problem is the export step in the pipeline. It creates weird artifacts on water areas that then result in errors during unwrapping: The errors are gone using Snaphu Export in SNAP-6, which I have installed locally. I found this thread on the STEP forum that might be related, but I am not sure as if this is the case. My problem is I cannot really test the Export in SNAP 8 Desktop since I don't have it locally. Thank you both! |
good stuff! Yes I have snap 8. I am going to another Oslo meeting for a few days now, but I will try it get it running tonight or tomorrow. Can snap 8 not be installed in the docker? |
Yes but only command line for now. I am trying to determine if this is a bug from snappy, from SNAP-8 or from any other source... That's why I want to run the process on Snap-8 Desktop |
Thanks Ani! Yes looks good so SNAP 6 and 7 work ok... let's see if SNAP-8 Desktop gives issues |
Looks great! Sorry I haven't had time today yet to check
Get Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Lorena Abad Crespo ***@***.***>
Sent: Wednesday, November 3, 2021 4:54:27 PM
To: SliDEM-project/SliDEM-python ***@***.***>
Cc: Benjamin Robson ***@***.***>; Mention ***@***.***>
Subject: Re: [SliDEM-project/SliDEM-python] Snaphu export fails for subsets with water in SNAP 8 (Issue #9)
Thanks Ani! Yes looks good so SNAP 6 and 7 work ok... let's see if SNAP-8 Desktop gives issues
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#9 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AMTZPBK6LYJ4MICMHV4ZT7TUKFLLHANCNFSM5GOD24CA>.
Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
No worries! take your time :) |
Maybe related but there have been several failures also with the message:
Some checks indicate that coherence is 0 everywhere, any thoughts @ZahraDabiri? |
When doing the unwrapping, with a single tile, Snaphu fails with the following:
Working on a subset inlands works good.
cc @bro076
The text was updated successfully, but these errors were encountered: