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

Support loading all Swaths for IW mode of Sentinel-1 #21

Open
whatnick opened this issue Jan 27, 2020 · 6 comments
Open

Support loading all Swaths for IW mode of Sentinel-1 #21

whatnick opened this issue Jan 27, 2020 · 6 comments

Comments

@whatnick
Copy link

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.

@birgander2
Copy link
Owner

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.

whatnick added a commit to whatnick/PyRAT that referenced this issue Jan 27, 2020
Signed-off-by: Tisham Dhar <whatnickd@gmail.com>
@whatnick
Copy link
Author

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.

whatnick added a commit to whatnick/PyRAT that referenced this issue Jan 27, 2020
Signed-off-by: Tisham Dhar <whatnickd@gmail.com>
whatnick pushed a commit to whatnick/PyRAT that referenced this issue Jan 27, 2020
@birgander2
Copy link
Owner

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?

@whatnick
Copy link
Author

whatnick commented Feb 7, 2020 via email

@birgander2
Copy link
Owner

I added a new branch with your changes and some additional fixes I did for dual-pol data.

@whatnick
Copy link
Author

whatnick commented Mar 2, 2020

Thanks @birgander2 been working on other stuff. Looks good ... suitable for merging to master ? Should I close my PR ?

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

2 participants