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

context extension vs numberMatched and numberReturned from OGC API #39

Closed
mzaglia opened this issue Aug 20, 2020 · 7 comments · Fixed by #80
Closed

context extension vs numberMatched and numberReturned from OGC API #39

mzaglia opened this issue Aug 20, 2020 · 7 comments · Fixed by #80
Assignees
Milestone

Comments

@mzaglia
Copy link

mzaglia commented Aug 20, 2020

Should I use the context extension instead of the OGC API equivalents?

@cholmes
Copy link
Collaborator

cholmes commented Nov 2, 2020

@mzaglia - apologies for the super slow response. We're at a weird state with these, as we're hoping the OGC adapts to our way, but their core stuff seems to be moving slowly. If you want to be fully in line with both you can implement both in the same response. Which is clearly a bit weird, but we hold hope OGC will update. You could do /search/ in STAC with STAC style and OGC API ones with their style. Sorry we don't have a better answer here.

@m-mohr
Copy link
Collaborator

m-mohr commented Nov 18, 2020

Would this get resolved with the new conformance classes? OGC way in /collections/... and context extension in /search?

@cholmes
Copy link
Collaborator

cholmes commented Dec 1, 2020

I wouldn't call that fully resolved, but yes, that should make things clearer. We can add a note in 'context' about how we expect this just to be used with STAC search.

cholmes added a commit that referenced this issue Dec 1, 2020
@m-mohr m-mohr added this to the 1.0.0-beta.1 milestone Dec 1, 2020
@m-mohr
Copy link
Collaborator

m-mohr commented Dec 4, 2020

Maybe phrase it differently. It says now:

currently scoped to just the STAC Search functionality.

But it actually is scoped to be used with everything STAC specific (e.g. collection search, item search, maybe at some point other things), but not with OGC API features.

@cholmes
Copy link
Collaborator

cholmes commented Dec 4, 2020

But right now it's just item search, right? We don't have collection search yet or anything else. I was thinking that when a new thing come we'd add the scope there. I don't feel strongly, just felt a little more clear to say exactly what it does cover.

@m-mohr
Copy link
Collaborator

m-mohr commented Dec 7, 2020

It's the fragment and fragments by nature should not have a scope so that they can be re-used. Currently it sounds like it can't be re-used as it's only meant to be used with Item Search. (Actually, people may decide to use this fragment in Features for backward compatibility.)

I get what you want to express though... Maybe just start with something like that:

This extension is currently scoped to just the STAC-specific functionality such as STAC Search.

@cholmes
Copy link
Collaborator

cholmes commented Dec 7, 2020

Cool, I'll try again - I wasn't sure exactly how to talk about fragments / extensions well.

@m-mohr m-mohr linked a pull request Dec 7, 2020 that will close this issue
4 tasks
@cholmes cholmes mentioned this issue Dec 8, 2020
4 tasks
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.

3 participants