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.

Sign up for GitHub

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 in RDF dataset definitions from SPARQL query #50

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

add in RDF dataset definitions from SPARQL query #50

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
78cff60
add in RDF dataset definitions from SPARQL query
pfps Oct 31, 2024
70e6061
remove definition of active graph as it is SPARQL specific
pfps Oct 31, 2024
d1e7f07
fix definition markup
pfps Nov 1, 2024
13842cf
Update spec/index.html
pfps Nov 1, 2024
e6b5e69
Clean up "merge" definition to use the fragment "dfn-rdf-graph-merge"…
gkellogg Nov 2, 2024
52c0fae
Fix duplicate definitions of "RDF dataset".
gkellogg Nov 2, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
54 changes: 54 additions & 0 deletions spec/index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1498,6 +1498,60 @@ <h2>RDF Datasets</h2>
between datasets would require RDF graph entailments between the graphs with the same name
(adding in empty graphs as required).</p>

<table>
<caption>Definition of an RDF dataset</caption>
<tr>
<td class="semantictable">
<p>An <dfn data-lt="RDF dataset">RDF dataset</dfn> I is a set:</p>
Copy link
Member

Choose a reason for hiding this comment

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

RDF Dataset is actually defined in RDF Concepts, although it has an alias here, so it's sufficient to just use an anchor.

Suggested change
<p>An <dfn data-lt="RDF dataset">RDF dataset</dfn> I is a set:</p>
<p>An <a>RDF dataset</a> I is a set:</p>
```suggestion

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should be fixed in latest commit.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The dfn markup is copied from examples elsewhere. If they are wrong here then I think there are lots that have to be changed throughout the document set.

Copy link
Member

Choose a reason for hiding this comment

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

This document is more complicated than most as the form is to "define" terms which an also defined in RDF Concepts. The <dfn data-cite="...">...</dfn> form is there to be able to use these terms as if they're defined in RDF Semantics while referencing the version in RDF Concepts. Note that data-lt provides an alternative way to reference the same definition, but it can also affect the term which is "exported" from the document. It can be useful to look at the Terms defined by this specification section to see what terms are actually defined here.

I did scan the other definitions in the document and they seem correct to me.

{ G, (&lt;u<sub>1</sub>&gt;, G<sub>1</sub>), (&lt;u<sub>2</sub>&gt;, G<sub>2</sub>), . .
. (&lt;u<sub>n</sub>&gt;, G<sub>n</sub>) }<br>
where n&ge;0 and G and each G<sub>i</sub> are graphs, and each &lt;u<sub>i</sub>&gt; is an IRI. Each
&lt;u<sub>i</sub>&gt; is distinct.</p>
<p>G is called the default graph. (&lt;u<sub>i</sub>&gt;, G<sub>i</sub>) are called named
graphs.</p>
</td>
</tr>
</table>
<table>
<caption>Definition of active graph in an RDF dataset</caption>
<tr>
<td class="semantictable">
<p>The <dfn data-lt="active graph">active graph</dfn> in an RDF dataset is the graph from the dataset used for basic graph pattern
matching.</p>
</td>
</tr>
</table>
Copy link

Choose a reason for hiding this comment

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

This is SPARQL specific and can be left in SPARQL Query. It changes during query execution (GRAPH).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sounds reasonable. I put it in just in case but I'll take it out.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done


<table>
<caption>Definition of RDF dataset merge</caption>
<tr>
<td class="semantictable">
<p>Let DS1 = { G1, (&lt;u1<sub>1</sub>&gt;, G1<sub>1</sub>), (&lt;u1<sub>2</sub>&gt;,
G1<sub>2</sub>), . . . (&lt;u1<sub>n</sub>&gt;, G1<sub>n</sub>) },<br>
and DS2 = { G2, (&lt;u2<sub>1</sub>&gt;, G2<sub>1</sub>), (&lt;u2<sub>2</sub>&gt;,
G2<sub>2</sub>), . . . (&lt;u2<sub>m</sub>&gt;, G2<sub>m</sub>) }</p>
<p>The <dfn data-lt="RDF dataset">RDF dataset merge</dfn> of DS1 and DS2 is:<br>
gkellogg marked this conversation as resolved.
Show resolved Hide resolved
DS={ G, (&lt;u<sub>1</sub>&gt;, G<sub>1</sub>), (&lt;u<sub>2</sub>&gt;, G<sub>2</sub>), .
. . (&lt;u<sub>k</sub>&gt;, G<sub>k</sub>) }<br>
where:</p>
<p>Write N1 for { &lt;u1<sub>j</sub>&gt; j = 1 to n }<br>
Write N2 for { &lt;u2<sub>j</sub>&gt; j = 1 to m }<br></p>
<ul>
<li>G is the <a data-cite="RDF12-SEMANTICS#dfn-merge">merge</a> of G1 and G2
</li>
<li>(&lt;u<sub>i</sub>&gt;, G<sub>i</sub>) where &lt;u<sub>i</sub>&gt; is in N1 but not
in N2</li>
<li>(&lt;u<sub>i</sub>&gt;, G<sub>i</sub>) where &lt;u<sub>i</sub>&gt; is in N2 but not
in N1</li>
<li>(&lt;u<sub>i</sub>&gt;, G<sub>i</sub>) where &lt;u<sub>i</sub>&gt; is equal to
&lt;u<sub>j</sub>&gt; in N1 and equal to &lt;u<sub>k</sub>&gt; in N2 and G<sub>i</sub>
is the <a data-cite="RDF12-SEMANTICS#dfn-merge">merge</a> of G1<sub>j</sub> and G2<sub>k</sub>
</li>
</ul>
</td>
</tr>
</table>

</section>

<h2 id="appendices">Appendices</h2>
Expand Down
Loading