-
Notifications
You must be signed in to change notification settings - Fork 5
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
General:Start introducing language tags #30
Conversation
@@ -6,6 +6,8 @@ | |||
|
|||
public class SIB { | |||
public static final IRI DESCRIBE = SimpleValueFactory.getInstance().createIRI(SHACL.NAMESPACE, "describe"); | |||
public static final IRI BIGDATA_SELECT = SimpleValueFactory.getInstance().createIRI(SHACL.NAMESPACE, "bigdata_select"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need a separate predicate/class for blazegraph select?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because there are many "SPARQL" queries on WikiData that use Blazegraph extensions that are not at all SPARQL 1.1. i.e. we have users outside of SIB now for this code. But that should not have been in this PR. I have accidentally pushed that to this branch :(
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be fixed now with a force push
cbec538
to
e875c06
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Would be nice to add a line in the Readme about this so that future contributors are made aware of this.
@@ -5,11 +5,11 @@ | |||
|
|||
ex:11 a sh:SPARQLExecutable, | |||
sh:SPARQLSelectExecutable ; | |||
rdfs:label "Mapping HMDB to SwissLipids" ; | |||
rdfs:label "Mapping HMDB to SwissLipids"@en ; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do we have rdfs:label instead of schema:keywords? Do we use both, for different purposes? We should also document this.
We should tag, when possible the language a string/comment/label is in. To allow for labels in many languages.