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

Content types #2

Merged
merged 6 commits into from
Feb 23, 2021
Merged

Conversation

MattTennison
Copy link
Contributor

What?

Add content type to MUD Vocab

Why?

To support changes in Multi-User-Domain/mud-jena#7

To account for non-sensual descriptions and support future enhancements that
wouldn't fit the senses model
mud.ttl Outdated
Comment on lines 106 to 113
mud:content a rdfs:Class ;
rdfs:label "Textual Content"@en ;
rdfs:comment "Content describing another model"@en ;

mud:sight a rdfs:Property
rdfs:label "Sight"@en ;
rdfs:comment "The visual sense"@en ;
rdfs:domain mud:content
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we move the new things into a new ontology please ? I'm thinking mudcontent ?

Just for modularity - if someone was writing their own front-end they might want to replace our definitions for the user's perception

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done in 1a6f286 😄

I then updated the Java app here Multi-User-Domain/mud-jena@59a821a

Copy link
Member

@calummackervoy calummackervoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

congrats on publishing your first ontology 😉

@@ -0,0 +1,25 @@
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .
@prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#> .
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added some extra imports. Note that whenever you use the syntax mud:something it's actually shorthand for the full URI https://raw.githubusercontent.com/Multi-User-Domain/vocab/main/mud.ttl#something

The prefix statements define what the shorthand is for

mudcontent:sight a rdfs:Property
rdfs:label "Sight"@en ;
rdfs:comment "The visual sense"@en ;
rdfs:domain mud:content .
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added a missing '.' character here. Every turtle statement ends in a full stop, the semi-colon is a break to say "done with this property, but the next one is about the same subject (mudcontent:sight)

@calummackervoy calummackervoy merged commit 7a74d9d into Multi-User-Domain:main Feb 23, 2021
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 this pull request may close these issues.

2 participants