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

Issue with tutorials/detecting-changes-in-sentinel-1-imagery-pt-2/index.ipynb #444

Closed
guiattard opened this issue Jan 21, 2022 · 2 comments
Assignees
Labels
enhancement New feature or request tutorial This issue or pull request is related to tutorials

Comments

@guiattard
Copy link
Contributor

guiattard commented Jan 21, 2022

Hello everyone,

I've been through the the python tutorial series focusing on sentinel-1. Thanks to the contributors and @mortcanty for this detailed and very interesting work. I would only have one recommandation to make easier the adaptation of the tutorial (part 2) to a user-specific area of interest (and different from the site proposed in the tutorial).

At the beginnig of the tutorial, the area of interest is defined as a polygon and just after that, the S1 image collection is filtered accordingly. In this procedure, the relative orbit number of interest (equal to 15) is specified but seems to come out of nowhere (see below).

im_coll = (ee.ImageCollection('COPERNICUS/S1_GRD_FLOAT')
                .filterBounds(aoi)
                .filterDate(ee.Date('2020-08-01'),ee.Date('2020-08-31'))
                .filter(ee.Filter.eq('orbitProperties_pass', 'ASCENDING'))
                .filter(ee.Filter.eq('relativeOrbitNumber_start', 15))
                .sort('system:time_start'))

To people not very familiar with the S1 product, it might be confusing and hard to understand how to adapt the number to the actual area of interest. Hence, I would recommand to add a line or a link to give an indication in that sence.

For example, I think that a reference to the Acquisition Plan Viewer might be helpfull to find relative orbit number associated to satellites, period and locations of interest.

Otherwise, the S1 series is great =)

Best regards,

@jdbcode jdbcode self-assigned this Feb 15, 2022
@jdbcode jdbcode added enhancement New feature or request tutorial This issue or pull request is related to tutorials labels Feb 15, 2022
@jdbcode
Copy link
Member

jdbcode commented Nov 30, 2023

Good suggestion. I wonder if it is necessary? It seems not. There is a date and aoi filter, which seems sufficient. I'll try without the relativeOrbitNumber_start and see if a different set of images are provided. In any case, it seems unnecessary for the purpose of the tutorial.

@jdbcode
Copy link
Member

jdbcode commented Dec 21, 2023

Best to keep the filter for relative orbit. I added a note where they can find relative orbit, see #768 (thanks for the suggestion about Acquisition Plan Viewer!)

@jdbcode jdbcode closed this as completed Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request tutorial This issue or pull request is related to tutorials
Projects
None yet
Development

No branches or pull requests

2 participants