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

Adds a definition for "IRI reference". #57

Merged
merged 4 commits into from
Aug 17, 2023
Merged

Adds a definition for "IRI reference". #57

merged 4 commits into from
Aug 17, 2023

Conversation

gkellogg
Copy link
Member

@gkellogg gkellogg commented Aug 9, 2023

We had defined "IRI" and "relative IRI reference", but not "IRI reference. This allows other specs to refer to RDF Concepts for all such terms.

For #20.


Preview | Diff

We had defined "IRI" and "relative IRI reference", but not "IRI reference. This allows other specs to refer to RDF Concepts for all such terms.
@gkellogg gkellogg added the spec:editorial Minor issue or proposed change in the specification (markup, typo, informative text) label Aug 9, 2023
spec/index.html Show resolved Hide resolved
spec/index.html Outdated Show resolved Hide resolved
spec/index.html Outdated Show resolved Hide resolved
Co-authored-by: Andy Seaborne <andy@apache.org>
spec/index.html Outdated Show resolved Hide resolved
@gkellogg gkellogg requested a review from afs August 10, 2023 15:51
spec/index.html Outdated
@@ -461,7 +461,7 @@ <h2>RDF Documents and Syntaxes</h2>
many different ways to encode the same <a>RDF graph</a> or
<a>RDF dataset</a>, for example through the use of
<a>namespace prefixes</a>,
<a>relative IRI references</a>, <a>blank node identifiers</a>,
<a>IRI references</a>, <a>blank node identifiers</a>,
and different ordering of statements. While these aspects can have great
Copy link
Contributor

Choose a reason for hiding this comment

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

Unrelated: should that be "triples"?

Copy link
Member

Choose a reason for hiding this comment

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

@afs -- Which "that"?

Copy link
Contributor

Choose a reason for hiding this comment

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

"statements"

Copy link
Member

Choose a reason for hiding this comment

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

I wouldn't argue with changing ordering of statements but given that this is talking about ways to encode the same <a>RDF graph</a> or <a>RDF dataset</a>, I think it must become ordering of triples and/or quads or possibly ordering of triples (and/or quads).

I see no strong reason not to make that tiny additional change within this PR, but if anyone objects to that, I ask that you create an issue referencing this discussion.

Copy link
Member Author

Choose a reason for hiding this comment

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

Changed to "triples". "quads" aren't appropriate in this context, and N-Quads actually uses "statements" in some circumstances where N-Triples uses "triples". In this context, "triples" is correct.

Copy link
Contributor

Choose a reason for hiding this comment

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

That would be tricky. Quads only appear in non-normative 4.3.

The definition of RDF Datasets is about graphs, and those graphs are sets of triples.

spec/index.html Outdated Show resolved Hide resolved
@gkellogg gkellogg merged commit 664a651 into main Aug 17, 2023
1 check passed
@gkellogg gkellogg deleted the iri-reference branch August 17, 2023 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec:editorial Minor issue or proposed change in the specification (markup, typo, informative text)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants