PyRAT
PyRAT copied to clipboard
birgander2
Support loading all Swaths for IW mode of Sentinel-1
Currently PyRAT only loads IW1 ( first swath of interferometric wide-swath mode). Extend this function to load all swaths in this mode into 6 separate layers instead of 2.
Mmh, I need such a data set, have to ask some colleagues. Generally, I'm not happy how gdal handles S-1 data. Correct would be one (or 2) layers containing the entire swath. But gdal doesn't support linking the swaths.
I have created PR #22 to address this and successfully tested it. The notebook linked in the PR also contains some Australian sample datasets I am analyzing for fire activity.
This needs to be improved a bit. You don't always have 3 subswaths, and in such a case the module crashes if you select the a wrong swath. Also, the selector in the gui might be misleading, as at that point you don't know yet the number of swaths.
I tend to replace this option by a switch "first / all swaths". What do you think?
Sounds good. I will make necessary changes in the patch and perhaps add a note in the code with link to all of the Sentinel 1 imaging modes.
Most of the data available to me is IW mode with 3 sub-swaths. Don't have access to other data unfortunately.
On Fri, Feb 7, 2020, 20:52 Anderl [email protected] wrote:
This needs to be improved a bit. You don't always have 3 subswaths, and in such a case the module crashes if you select the a wrong swath. Also, the selector in the gui might be misleading, as at that point you don't know yet the number of swaths.
I tend to replace this option by a switch "first / all swaths". What do you think?
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/birgander2/PyRAT/issues/21?email_source=notifications&email_token=AADX7BDFVPVZGLSB7O76ZATRBUVOZA5CNFSM4KL573MKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELCLDYQ#issuecomment-583315938, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADX7BGDSZMQMHOZUUEHO2TRBUVOZANCNFSM4KL573MA .
I added a new branch with your changes and some additional fixes I did for dual-pol data.
Thanks @birgander2 been working on other stuff. Looks good ... suitable for merging to master ? Should I close my PR ?