-
Notifications
You must be signed in to change notification settings - Fork 34
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
SLC Incorrect Data on Write #502
Comments
Sounds BAD. Will try to fix. But can you download the file inline in your MWE? Otherwise a big chunk of my available bugfix time dissapears looking for demo rasters on various platforms, rather than fixing bugs ;) |
I know it's really annoying to reproduce...I don't think they have an API though. They require an account and all the files are > 1Gb. My hunch is that is has to do with rotations, but I'll try to get a simpler MWE tomorrow |
I just couldnt find the file after 5 mins looking a gave up 😂 |
I found a file matching that name on there but it won't even load in GDAL so I'm not sure what you're actually doing - I really need a full MWE here |
Okay, here's some better info. @rafaqz I sent you a link to download the data. I don't think any SLC data exists in a smaller file, so that's annoying. I can't give you a smaller file but here's some simplified code that identifies the issue: using Rasters, ArchGDAL
path = "s1a-iw1-slc-vh-20221206t234001-20221206t234026-046218-0588c1-001.tiff"
a = Raster(path)
Rasters.write("temp.tiff", a; force=true)
b = Raster("temp.tiff") # roundtrip
a.data == b.data # false
a.data == reverse(b.data, dims=2) # true |
Ok, this is kind of minor problem really. By convention non rotated geotiffs are usually stored with the Y axis flipped, or "north up". So users don't have to do this reversing manually all the time we do it automatically. I guess if there is any kind of rotation we should'nt do that, instead just leaving things how they are. This is pretty well explained in this stackexchange (Thanks for the drive link its super fast to check these things with a file and MWE) |
I have a Single Look Complex .tiff file where the underlying data changes on a read-write-read roundtrip.
Data found on https://search.asf.alaska.edu/ (S1A_IW_SLC__1SDV_20230103T230613_20230103T230641_046626_059699_9E8C)
A likely related issue is that the written file is about half the size as the original:
I'll look into this further and add more details as I find them. Just wanted to get an issue in writing.
The text was updated successfully, but these errors were encountered: