-
Notifications
You must be signed in to change notification settings - Fork 3
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
Scenario Ontology: Link to OpenDRIVE map #47
Comments
Yes, we thought of that too. And we would also prefer to use the DID as a link. However, we weren't sure if According to https://www.ietf.org/rfc/rfc2396.txt I found the following regex for URI From this I would conclude that this is okay then? Are there any other opinions? Othewise I would change it and create a pull request. |
@robertschubert what is your opinion? It makes sense to me. What did method should we use for assets? |
I just talked wit @lenasauermann about it. I'll create a new branch and use a DID. |
The other question is if we want to define "asset bundles" in a generic way. So remove these nested links and remove a separate ontology for a bundle? |
I'm not sure about that one. Last time we discussed this, we concluded that we will not use bundles inside our ontology. The ontology however, should offer the oppertunity to modell required and optional "dependencies" or links. A data provider/marketplace can then offer a bundle based on these dependencies. However, we can reopen that discussion. |
I am ok if you discussed it! |
@lenasauermann I think you missed the change here: https://github.com/GAIA-X4PLC-AAD/ontology-management-base/blob/main/scenario/scenario_shacl.ttl |
You mean the example in there? https://github.com/GAIA-X4PLC-AAD/ontology-management-base/blob/main/scenario/scenario_shacl.ttl#L207C23-L207C54 |
I am looking at the following element:
I think a map is its own asset with its own meta data description. In this case we would depend with a scenario on a map that has already been instantiated as a separate asset and should be referenced through the asset id?
Maybe @lenasauermann has an opinion here as well?
The text was updated successfully, but these errors were encountered: