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

Number of boxes in Extent → Spatial → bbox #31

Open
ways opened this issue Nov 18, 2024 · 2 comments
Open

Number of boxes in Extent → Spatial → bbox #31

ways opened this issue Nov 18, 2024 · 2 comments
Assignees

Comments

@ways
Copy link
Collaborator

ways commented Nov 18, 2024

There's some confusion whether bbox should consist of an array, or an array of arrays.

Single bbox:

"extent": {
        "spatial": {
          "bbox": [
            -180.0,
            -89.9,
            180.0,
            89.9
          ],
          "crs": "..."
        },

Multiple bboxes:

"extent": {
        "spatial": {
          "bbox": [
            [
              -180.0,
              -89.9,
              180.0,
              89.9
            ],
          ],
          "crs": "..."
        },

According to opengeospatial/ogcapi-environmental-data-retrieval#331 originally option number 1 was correct, but later versions allowed option number two, with the warning that client supporting core could break.

I think it's important we choose one of these, not both.

@havardf
Copy link
Collaborator

havardf commented Nov 18, 2024

The description in the OpenAPI specification (https://github.com/opengeospatial/ogcapi-environmental-data-retrieval/blob/master/core/standard/openapi/schemas/collections/extent.yaml) says this about the bbox:

One or more bounding boxes that describe the spatial extent of the dataset. In the Core only a single bounding box is supported. Extensions may support additional areas. If multiple areas are provided, the union of the bounding boxes describes the spatial extent.

I struggle to understand why this is allowed, and why not just make the union of multiple areas into one bigger bbox in the first place.

However, if the profile say array of numbers instead of array of array of numbers, then we could cause problems for client libraries. So it seems to me that the best we can do is to say its an array of array with min and max values equal to 1?

@geira
Copy link

geira commented Nov 28, 2024

I struggle to understand why this is allowed, and why not just make the union of multiple areas into one bigger bbox in the first place.

Consider the US of A, it makes more sense to have one bounding box for the 48 contiguous states and others for Hawaii and Alaska, respectively.

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

3 participants