Skip to content
This repository has been archived by the owner on Feb 15, 2020. It is now read-only.

Added compound literals #23

Merged
merged 4 commits into from
Aug 29, 2019
Merged

Added compound literals #23

merged 4 commits into from
Aug 29, 2019

Conversation

iherman
Copy link
Member

@iherman iherman commented Aug 27, 2019

I picked up #22 and added to the document as yet another, possible solution; thanks to @afs

@afs can you look at this to see if it makes sense?


Preview | Diff

@iherman iherman requested a review from pchampin August 27, 2019 15:31
@afs
Copy link

afs commented Aug 28, 2019

Looks good! Thanks @iherman

Copy link
Collaborator

@pchampin pchampin left a comment

Choose a reason for hiding this comment

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

I like the text as it is (except for the parenthesis about mixing langString and compount literals, but that is a detail).

However, it would seem clearer to include it in the "RDF based solutions" section. After all, it is purely RDF based. The current solutions in this section leverage.

@iherman if you agree, I can make the change.

index.html Outdated
</p>

<p>
(It would be possible to make a “mix” whereby, instead of using <code>rdf:language</code>, one would use a <code>rdf:langString</code> as an object for <code>rdf:value</code>.)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Yuk :-/ I am not sure I want to hint at that solution... IMO, it has the worse of both worlds: there is still an indirection to follow to "find" the actual string, without the benefit of cleanly separating its metadata in distinct triples... Worth, the metadata are heterogeneous in that respect.

I suggest we remove this parenthesis.

Copy link
Member Author

Choose a reason for hiding this comment

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

I am not saying I like it... :-)

I am fine removing it, but you can bet someone will raise this!

@iherman
Copy link
Member Author

iherman commented Aug 29, 2019

I am fine moving it to the RDF section. I would appreciate you do it:-)

@pchampin
Copy link
Collaborator

I would appreciate you do it:-)

Done. Please have a look at the revise "General comments: Pros and cons" at the end of section 2.1 . I had to modify it slightly to account for the new RDF-based solution.

@iherman
Copy link
Member Author

iherman commented Aug 29, 2019

I changed only one thing: by adding this solution to the lot, it may not be necessarily the case that a new WG is required to do this. Defining 3-4 extra terms may be done in another structure; e.g., I could imagine the I18N WG defining, formally, those terms. (Or may be even not requiring a WG but only a CG; to be discussed.)

I have somewhat weakened the last sentence, and I will now merge it. Discussions with this group to follow...

@iherman iherman merged commit acee948 into master Aug 29, 2019
@iherman iherman deleted the extra-terms branch August 29, 2019 13:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants