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

Multiple unnecessary schema redefinition #320

Closed
fmigneault opened this issue Feb 25, 2023 · 2 comments
Closed

Multiple unnecessary schema redefinition #320

fmigneault opened this issue Feb 25, 2023 · 2 comments

Comments

@fmigneault
Copy link
Member

Multiple schemas are redundant under: https://github.com/opengeospatial/ogcapi-processes/tree/master/openapi/schemas

Examples:

Items that can be reused should do so to avoid diverging definitions and interpretation by developers/users/implementations.

Furthermore, reusing the https://github.com/opengeospatial/ogcapi-common schemas would facilitate interoperability between the various reusable components under the "OGC API - {standard}" umbrella.

@jerstlouis
Copy link
Member

jerstlouis commented Feb 25, 2023

@fmigneault

Items that can be reused should do so to avoid diverging definitions and interpretation by developers/users/implementations. Furthermore, reusing the https://github.com/opengeospatial/ogcapi-common schemas would facilitate interoperability between the various reusable components under the "OGC API - {standard}" umbrella.

I fully agree with the goals here.

However, for various reasons such as the ability to include things in metanorma documents, easily produce an API definition with swagger-cli bundle without external dependencies, and having normative schemas at https://schemas.opengis.net/ogcapi/processes/ before OGC API - Common is actually published, the building blocks originating from elsewhere are included here.

The idea to organize the building blocks into /common-* is precisely to try to facilitate synchronizing those builing blocks (e.g., by doing a directory/files comparison only within that sub-directory).

I hope the efforts with the building blocks register and overall efforts to improve semantic support in the standards and the definition server will make this easier and help achieve and maintain consistency.

We adopted the same practice in OGC API - Tiles, DGGS, Coverages, Maps, and those common- blocks should be mostly in sync there. The Common group has not met in a while, but this is related to opengeospatial/ogcapi-common#302.

@fmigneault
Copy link
Member Author

@pvretano
I do not think this blocks the standard definition.
It's something that should be improved over time using the building blocks and the official schema references rather than the local copies to help cross OGC API references to work as intended.
Currently, there are potential discrepancies between published schemas and the local ones.

I will close so that we focus on "more important" issues. We can reopen once building blocks will be better defined.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants