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

search_data results for OPERA_L2_CSLC-S1_V1 are much larger than expected #746

Closed
scottstanie opened this issue Jul 5, 2024 · 3 comments

Comments

@scottstanie
Copy link

I was checking how hard it woudl be to use earthaccess to download some OPERA CSLC products (taking the small search from here as an example)

import earthaccess
earthaccess.login()

results = earthaccess.search_data(
    short_name='OPERA_L2_CSLC-S1_V1',
    cloud_hosted=True,
    bounding_box=(-117.33, 35.541, -117.880, 35.991),
    temporal=("2023-10-01", "2023-10-15"),
    count=500
)
Granules found: 2737

This is a small bounding and only 15 days, so I'd expect just a few results.

Entering this into ASF (or using asf_search) yields the expected 12 granules

I'm not sure if this is something odd or unexpected with the OPERA data compared to others, or how earthaccess would match this many granules.

@scottstanie
Copy link
Author

Shoot I'm sorry, I just copied those parameters to test, but I didn't notice that the bounding box from that issue was incorrect and had the order (right, bottom, left, top).

Using bounding_box=(-117.33, 35.541, -117.880, 35.991) returns 10 results.

I actually saw this because the asf_search library printed the helpful warning

WARNING:asf_search:WKT REPAIR/VALIDATION: The following repairs were performed on the provided AOI:
["'type': 'REVERSE': 'report': Reversed polygon winding order"]

Not sure if that's worth adding here or not, but it helped me pick out that problem.

@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in earthaccess project Jul 5, 2024
@mfisher87 mfisher87 closed this as not planned Won't fix, can't repro, duplicate, stale Jul 5, 2024
@mfisher87
Copy link
Collaborator

I really appreciate you sharing the problem solving process! It may help someone else some day :) ❤️

@jhkennedy
Copy link
Collaborator

@scottstanie it might be worth opening a feature request for python-cmr to do the same validation of geometries as asf_search.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

3 participants