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

Recognize rdf:dirLangString #64

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Recognize rdf:dirLangString #64

wants to merge 1 commit into from

Conversation

afs
Copy link

@afs afs commented Dec 21, 2024

This closes #59.
Replacement for PR #60.

This PR includes updates for the "Semantic conditions for literals table" and "RDFS semantic conditions".


Preview | Diff

Copy link
Member

@gkellogg gkellogg left a comment

Choose a reason for hiding this comment

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

On tiny grammar change. Otherwise, looks good.

spec/index.html Outdated Show resolved Hide resolved
@afs afs force-pushed the three-special-datatypes branch from 47b1510 to 6cf8ab8 Compare December 21, 2024 20:25
@afs afs requested a review from gkellogg December 21, 2024 20:25
@pfps pfps added the spec:enhancement Change to enhance the spec without affecting conformance (class 2) –see also spec:editorial label Dec 21, 2024
Copy link
Contributor

@pfps pfps left a comment

Choose a reason for hiding this comment

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

I believe that there needs to be a WG decision to add this. I created w3c/rdf-star-wg#139 for this purpose.

(The request changes flag is only to prevent yet another rogue merge.)

@afs
Copy link
Author

afs commented Dec 22, 2024

I believe that there needs to be a WG decision to add this.

There has been this WG resolution

support initial text direction in RDF Language-Tagged Literals

and this WG resolution

Add base direction as a forth element of Literals

and this WG resolution

Accept RDF-Concepts issue "text direction"


RDF 1.1 Semantics has it's own description of datatypes - it needs updating.

Why is rdf:langString required?

RDF 1.1 Semantics calls out rdf:langString as a normative "exception" because it has a special value mapping from RDF term to value. It gives a fixed (normative) mechanism for rdf:langString that includes handling the language tag in a case insensitive manner. Semantic extensions must agree on this exception.

RDF 1.2 Semantics must do so for rdf:dirLangString with a compatible mechanism, adding dealing with base direction. This PR uses the same text wording style to emphasis this.

The same is true for rdf:dirLangString in order to enact the WG resolutions, making it available for semantic extensions with the same interpretation for all semantic extensions.

Otherwise, the text of RDF Semantics need significant revision because it talks in general terms about the datatype mapping lexical space to value space.

@pfps
Copy link
Contributor

pfps commented Dec 22, 2024

Part of this PR is

RDF processors are not required to recognize any datatype IRIs other than
xsd:string,
rdf:langString, and
rdf:dirLangString
I don't believe that the requirement to recognize directional language strings has been decided.

@afs
Copy link
Author

afs commented Dec 22, 2024

Why is rdf:langString required?

@gkellogg
Copy link
Member

I believe the need to support rdf:dirLangString semantics is implicit in the other approvals. Indeed, if it were not a recognized datatype with conditions based on rdf:langString it would be inconsistent. The Working Group's time is quite limited, and easily gets taken up on perm-threads. No need to add further burden to go over something that is so clearly necessary. If there are reasonable objections, they can be made here, or in #60.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec:enhancement Change to enhance the spec without affecting conformance (class 2) –see also spec:editorial
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Updates for rdf:dirLangString
3 participants