Navigation Menu

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

Core refers to bbox-crs #454

Closed
m-mohr opened this issue Nov 18, 2020 · 1 comment · Fixed by #661
Closed

Core refers to bbox-crs #454

m-mohr opened this issue Nov 18, 2020 · 1 comment · Fixed by #661
Labels
Part 1: Core Issue related to Part 1 - Core

Comments

@m-mohr
Copy link
Contributor

m-mohr commented Nov 18, 2020

The Core specification refers to a parameter called bbox-crs in several places, but it's not defined anywhere in the Core spec. I'm personally aware that it's likely defined in the CRS extension, but the average reader probably not. So either it should be removed from Core or Core should make clear in the spec that this is a parameter not available unless the CRS extension is implemented.

m-mohr added a commit to radiantearth/stac-api-spec that referenced this issue Nov 18, 2020
@cportele
Copy link
Member

The text currently states:

This standard does not specify requirements for the parameter bbox-crs. Those requirements will be specified in an additional part of the OGC API Features series.

But this is stated only once and not everywhere bbox-crs is mentioned.

it's likely defined in the CRS extension

It is. With part 2 published, we could also add a reference to the bbox-crs definition.

@cportele cportele added the Part 1: Core Issue related to Part 1 - Core label Nov 21, 2020
@cportele cportele added this to Backlog in Part 1: Core via automation Nov 21, 2020
@cportele cportele moved this from Backlog to In Progress in Part 1: Core Nov 21, 2020
@cportele cportele moved this from In Progress to To be drafted in Part 1: Core May 10, 2021
cportele added a commit that referenced this issue Jan 3, 2022
@cportele cportele moved this from To be drafted to In Review in Part 1: Core Jan 3, 2022
Part 1: Core automation moved this from In Review to Done Jan 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Part 1: Core Issue related to Part 1 - Core
Projects
Part 1: Core
  
Done
Development

Successfully merging a pull request may close this issue.

2 participants