Skip to content

Commit e16dda5

Browse files
committed
adds issue notes for he issues discovered while working on this PR
1 parent 51285a1 commit e16dda5

File tree

1 file changed

+13
-0
lines changed

1 file changed

+13
-0
lines changed

spec/index.html

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9167,6 +9167,10 @@ <h4>Examples of Mapped Graph Patterns</h4>
91679167
BGP( ?s :p1 ?v1 . ?s :p2 ?v2 )
91689168
</div>
91699169
</div>
9170+
<div class="issue" data-number="246">
9171+
The example above does not include the version of the resulting expression
9172+
before the simplification step. The same is true for several other examples
9173+
below. The examples should be consistent in this regard.</div>
91709174
<p>Example: group consisting of a union of two basic graph patterns:</p>
91719175
<div class="algExample">
91729176
<div class="algExample1">
@@ -9286,6 +9290,11 @@ <h4>Examples of Mapped Graph Patterns</h4>
92869290
</div>
92879291
<p>Example: Pattern involving BIND, with a
92889292
<a href="#sparqlSimplification">simplification</a> step:</p>
9293+
<div class="issue" data-number="246">
9294+
The following example uses <code>{}</code> to represent the empty BGP in the
9295+
first version of the resulting expression (the one before the simplification
9296+
step), whereas the previous examples above are using |Z| instead. The
9297+
examples should be consistent in this regard.</div>
92899298
<div class="algExample">
92909299
<div class="algExample1">
92919300
{ ?s :p ?v . {} BIND (2*?v AS ?v2) }
@@ -9366,6 +9375,8 @@ <h5>Grouping and Aggregation</h5>
93669375
The transformation for query levels that use any aggregates is given
93679376
below:
93689377
</p>
9378+
<div class="issue" data-number="247">
9379+
There are a few minor issues in the following algorithm.</div>
93699380

93709381
<pre class="code nohighlight">Let <var>A</var> := the empty sequence
93719382
Let <var>Q</var> := the query level being evaluated
@@ -9551,6 +9562,8 @@ <h5>OFFSET and LIMIT</h5>
95519562
<p>|length| defaults to (size(|M|)-|start|).</p>
95529563
</blockquote>
95539564
</blockquote>
9565+
<div class="issue" data-number="248">
9566+
The definition of this translation step is formally incorrect.</div>
95549567
</section>
95559568
<section id="sparqlAlgebraOutcome">
95569569
<h5>Final Algebraic Query Expression</h5>

0 commit comments

Comments
 (0)