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

Add hydra:Rfc6570Template. #87

Merged
merged 1 commit into from Jan 19, 2015
Merged

Add hydra:Rfc6570Template. #87

merged 1 commit into from Jan 19, 2015

Conversation

RubenVerborgh
Copy link
Contributor

Implements the proposed resolution for #17.

"comment": "A templated string with placeholders. The literal's datatype indicates the template syntax; if not specified, hydra:rfc6570Template is assumed.",
"seeAlso": "hydra:rfc6570Template",
"domain": "hydra:IriTemplate",
"range": "rdfs:Literal",
Copy link
Member

Choose a reason for hiding this comment

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

I would prefer to not limit this quite generic property hydra:IriTemplate. Can we drop the domain (and possibly also the range) for the time being or use schema.org's counterparts instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agree on dropping the domain.
I strongly recommend to keep the range. All this says is that it's a literal property, and I know only one case where it makes sense to allow both objects and literals as a property's value (rdf:object).

Amended as 6df5dfc.

Copy link
Member

Choose a reason for hiding this comment

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

Really? Dropping the range would allow us to support templates which aren’t expressed in the form of a string (which needs to be parsed) but as a “structured value” which might be easier to process in a client.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Mwah, I think that's already the job of hydra:IriTemplate itself. The hydra:template property really points to a skeleton. Syntaxes can differ, but they will be strings.

Copy link
Member

Choose a reason for hiding this comment

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

Not necessarily I’d say. What can machines do in practice if we add this range that they can't otherwise?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Assume it's a literal property, and thus perform literal operations. Although it's probably mostly handy for humans writing program code against this property.

While I do think properties should at least be either literals or objects, I'm not fighting for this one. Whatever resolution you prefer is fine with me.

Copy link
Member

Choose a reason for hiding this comment

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

I would like to keep as many doors open as possible to allow usages we haven't anticipated. We can refine this later if necessary. So I'd suggest to drop the range for the time being.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Alright; amended to 7122446.

Copy link
Member

Choose a reason for hiding this comment

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

Great! Thanks. So the only remaining question is the capitalization. I’ll send out a straw poll to move this forward.

@lanthaler
Copy link
Member

I think we actually never discussed the capitalization of datatypes such as hydra:rfc6570Template. Should we keep it in lowercase or capitalize it as we do with classes, i.e., call it hydra:Rfc6570Template? Most XSD datatypes are lowercase, Schema.org's are all uppercase as are rdf:HTML and rdf:XMLLiteral.

Thoughts? Preferences?

@RubenVerborgh
Copy link
Contributor Author

I'd be included to keep it as-is; or either RFC in all-caps. Half-caps don't work for an acronym IMHO.

@akuckartz
Copy link

My preference: "RFC6570Template".

But I suggest to discuss capitalization conventions in the broader Semantic Web and Linked Data communities.

@lanthaler lanthaler changed the title Add hydra:rfc6570Template. Add hydra:Rfc6570Template. Jan 18, 2015
@@ -50,6 +50,7 @@
"TemplatedLink": "hydra:TemplatedLink",
"IriTemplate": "hydra:IriTemplate",
"template": "hydra:template",
"rfc6570Template": "hydra:rfc6570Template",
Copy link
Member

Choose a reason for hiding this comment

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

Please update the capitalization

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Amended in 690c4f2.

Implements the proposed resolution for #17.
lanthaler added a commit that referenced this pull request Jan 19, 2015
Add hydra:Rfc6570Template

This closes #17
@lanthaler lanthaler merged commit 358c0e1 into master Jan 19, 2015
@lanthaler lanthaler deleted the rfc6570template branch January 19, 2015 21:16
lanthaler added a commit that referenced this pull request Jan 19, 2015
@lanthaler
Copy link
Member

Thanks Ruben

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.

None yet

3 participants