-
Notifications
You must be signed in to change notification settings - Fork 2
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
Align on the usage of the relationships in the links section #67
Comments
Tagging @jdries @soxofaan @p3dr0 @HerveCaumont to get your feedback on this. |
Some considerations to take into account regarding the examples:
|
I've made some final updates to the relationship types. Unless there are any big objections, I suggest implementing them to close this topic. |
about this part of the proposal:
this obfuscates the fact that it's a link to an openEO UDP (originally/currently it's "rel=openeo-process"). Is this intentional, e.g. to make it generic/flexible? Note there are not really standard indicators that a json doc is intended to be an openEO UDP representation, so by removing that information from the relationship, things might get quite fuzzy and muddy |
Thank you for the feedback @soxofaan!
Correct, we want to align the relationship types to make them more generic and flexible. The details should be more reflected in the MIME type. However, you have a valid comment that json might be a bit too generic. Would it be an option to use |
related discussion: So with current state of that ticket I think we could go for |
Please consider Open-EO/openeo-api#532 (comment) before moving forward. General feedback:
|
This was also something we were talking about. Initially this was set to
The types are just examples based on the current content of the catalogue. The goal is to restrict the
Good suggestion |
Two words are fine, even offical IANA rel types have multiple words, e.g. |
regarding the mime types as @JanssenBrm mentioned we were considering this "as examples" for all relations but this reminds me that we forgot to consider their cardinality |
I understand your concern about the risks of confusion in existing clients when deviating from |
I don't know the exact context and the plans (i.e. didn't know about "developing new documents and new tools"), I just want to ensure we don't face issues in the core openEO tooling. :-) |
I think your point is valid @m-mohr to take the openEO tooling into account. However, we are currently not interacting with any openEO tools (AFAIK), so it could make sense to update our catalogue for now with the suggestion and come back to this if openEO tooling would be required or the final decision on the type has been made. |
Fair enough. I'd propose to NOT use The new proposal in Open-EO/openeo-api#532 (comment) is much easier to register with IANA as it just requires one registration instead of multiple registrations. So instead of I just want to ensure that we think about this enough before locking in on anything that may be an issue later. |
We should clearly define and align the relationship type in the links section used for openEO and OGC Processes. This would help streamline the visualization of the record in the Algorithm Catalogue.
Currently, we are using the following. Some of them have a mixed usage:
A suggestion (open to feedback):
The text was updated successfully, but these errors were encountered: