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

Support directional language-tagged strings (rdf:dirLangString) #112

Open
afs opened this issue Jul 6, 2023 · 9 comments · May be fixed by #113
Open

Support directional language-tagged strings (rdf:dirLangString) #112

afs opened this issue Jul 6, 2023 · 9 comments · May be fixed by #113
Assignees

Comments

@afs
Copy link
Contributor

afs commented Jul 6, 2023

See RDF concepts.

See https://github.com/w3c/rdf-star-wg/wiki/Text-Direction-Proposal

@afs afs linked a pull request Jul 6, 2023 that will close this issue
@afs afs self-assigned this Jul 27, 2023
@afs
Copy link
Contributor Author

afs commented Sep 12, 2024

PR #153 includes the grammar changes for rdf:dirLangString.

@domel
Copy link
Contributor

domel commented Sep 12, 2024

I want to bring to your attention a minor yet notable inconsistency in the naming conventions used since RDF 1.1. This version introduced rdf:langString, replacing the previous rdf:LangString, which breaks the convention of naming classes and datatypes with capital letters. Furthermore, the recently introduced rdf:dirLangString also starts with a lowercase letter, reinforcing this inconsistency.

While the naming of rdf:dirLangString at least aligns with the style of rdf:langString, the overall approach seems to deviate from the previously established conventions. This lack of uniformity in capitalization could lead to confusion among users and developers working with RDF, who typically expect such naming conventions to be more consistent.

@TallTed
Copy link
Member

TallTed commented Sep 12, 2024

I think @domel's point qualifies as a bug.

rdf:LangString should not have been replaced by rdf:langString.

This change should be reverted (restoring rdf:LangString), and rdf:dirLangString should be replaced by rdf:DirLangString.

@gkellogg
Copy link
Member

RDF Vocab defines rdf:langString.

@domel
Copy link
Contributor

domel commented Sep 12, 2024

@TallTed rdf:langString was proposed in RDF 1.1 and defined in vocab. So the situation is hopeless if we want to maintain backward compatibility.

@afs
Copy link
Contributor Author

afs commented Sep 12, 2024

This version introduced rdf:langString, replacing the previous rdf:LangString
I'm not sure what "This version" is referring to.

RDF 1.2 is not replacing anything from RDF 1.1.

previously established conventions

In RDF 1.1, there is rdf:HTML and rdf:XMLLiteral but the uppercase there is from the underlying name.

It's xsd:integer and xsd:dateTime. XSD dataypes are called out in RDF 1.1. So what convention there might be is quite weak.

@gkellogg
Copy link
Member

There are no syntaxes where you can use datatype rdf:langString or rdf:dirLangString, it's just for internal use. Normalizing to use these two variants instead of rdf:LangString or rdf:DirLangString should be a transparent change, and is necessary to fix an inconsistency. The fact that in RDF/SPARQL 1.1 no one noticed that it was capitalized is evidence that it doesn't matter to implementaitons.

@kasei
Copy link
Contributor

kasei commented Sep 12, 2024

Where was rdf:LangString previously defined?

There are no syntaxes where you can use datatype rdf:langString or rdf:dirLangString, it's just for internal use.

That may be true for RDF syntaxes, but it's not true in general. rdf:langString is returned by the DATATYPE function. Changing it would break queries.

@gkellogg
Copy link
Member

gkellogg commented Sep 12, 2024

Can someone help me find there the previous spec uses rdf:LangString? SPARQL Query Language for RDF doesn't mention it, and SPARQL 1.1 Query methods rdf:langString. I don't see any reference to rdf:LangString.

There's also this information box in 17.4.2.7:

The SPARQL Working Group is using rdf:langString based on the latest Working Drafts of the RDF Working Group. This usage should be considered experimental (and non-normative) until/unless rdf:langString becomes part of an updated RDF Recommendation.

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 a pull request may close this issue.

5 participants