-
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
OEM needs JSON-LD extension to support linking OEO entities #52
Comments
We talked over the metadata extension and to me it seems most suitable to add the currently proposed keys with tiny adaptions plus another addition. To summarize the "diff":
Sorry for the long comment, I'd appreciate feedback even for just smaller part of this. |
@jannahastings you commented in the issue linked above _OpenEnergyPlatform/oeo-extended#4 that there are pre-composition and post-composition approaches. I think my last suggestion would fall under the post-composition approach, which you advised against. Is it still useful to keep such a field for the occasional ontology nerd or should we drop the idea altogether? @0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q pointed out that we had a point on the two approaches on the agenda of an ontology developer meeting (namely # 16) but I can't quite figure out whether we got to it in any depth or postponed the topic without picking it up again. The notes in the wiki are inconclusive to me. |
From my understanding, both approaches are compatible with another and not exclusive. I can see ontology development being done by "experienced users" (or one-eyed kings with thick sunglasses among the blind, like me) in post-composition and integrated via pre-composition in the ontology proper after testing, discussion, and consensus. |
@christian-rli I think your proposal above sounds sensible, and I think for a use case linking such closely related projects such as the OEP - OEO, having a pipeline that supports post-coordination is absolutely sensible to avoid unneeded frustrating delays. Of course, as @0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q says, some of the post-coordinated expressions will make their way into the ontology class definitions over time. (I also think that the column name |
@yum-yab I think this identifier does not necessarily need to point to the databus file, right? or we could inject it when transforming the json to the RDF using the context. the connection via the databus file and the OEP metadata is given primarily already through the "Databus OEP annotation mod"?
@christian-rli I need a concrete example for this, for one column please (so is_about and value_reference filled out) |
I updated the example string with the help of @chrwm .
I'd like to discuss |
Make necessary changes to introduce the key is_about. The key should be used to describe the column header in ontology terms with a URI. See: #52
Make necessary changes to introduce the key value_reference. The key should be used to describe values in the column further, using ontology URI's. See: #52
Did the implemented new values meet the requirements from the OEO? |
@chrwm The issue will be closed as soon as the relase branch for version 1.5.0 is merged. Please let me know if you want the issue to stay open. |
Linking to entities in OEO should be supported by oemetadata. It is not clear how this is done best.
It is nice to have the discussion public for later reference and also to have it in one place. Since this is intended to end up in the metadata string, I suggest discussing and making changes in this issue, rather than at the original LOD-GEOSS location.
I added file originally commited by @yum-yab . Feel free to make changes in the opened branch add new files and so on. We can clean up the branch before merging or restart with a tidy branch later.
The text was updated successfully, but these errors were encountered: