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

ICESat2_Granule{:ATL06} are returned by find without bounding boxes. #28

Closed
alex-s-gardner opened this issue Oct 19, 2022 · 3 comments · Fixed by #38
Closed

ICESat2_Granule{:ATL06} are returned by find without bounding boxes. #28

alex-s-gardner opened this issue Oct 19, 2022 · 3 comments · Fixed by #38

Comments

@alex-s-gardner
Copy link
Collaborator

alex-s-gardner commented Oct 19, 2022

ICESat2_Granule{:ATL06} are returned by find without bounding boxes.

geotile = (min_x = 50.0, min_y = -88.0, max_x = 51.0, max_y = -87.0)
granules = find(:ICESat2, "ATL06", geotile, "005")

julia> granules[1].bbox
NamedTuple()

julia> granules[2].bbox
NamedTuple()

julia> granules[3].bbox
NamedTuple()

@evetion
Copy link
Owner

evetion commented Oct 19, 2022

Yeah, I designed those granules with the idea to hold extra metadata, but haven't implemented it, also because the metadata differs per product and is sometimes costly to get.

There's bounds that will calculate the bounding box based on the data, but that requires opening the (local) .h5 file (and while ICESat-2 stores the bbox in the metadata, GEDI doesn't, so we need to iterate through all coordinates to find the bbox).

There's also info, which parses the metadata from the filename.

I do see that there's a Polygon that's returned from NASA Earthdata Search for each granule, so we could use that as well.

@alex-s-gardner
Copy link
Collaborator Author

Given that the earthdata search quarry is so fast we could keep things light and just remove the bbox. I'm not using it in my processing. Do you have a need?

@evetion
Copy link
Owner

evetion commented Oct 23, 2022

I could use it, and it's faster than locally computing it. Also for saving all granules to a shapefile, this could work. Happy to make a PR for it.

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

Successfully merging a pull request may close this issue.

2 participants