- From: Gregory Williams <greg@evilfunhouse.com>
- Date: Tue, 6 Dec 2022 13:22:04 -0800
- To: public-rdf-dawg-comments@w3.org
- Message-Id: <23B81060-C6EC-49B6-991B-9A04F70D4D0C@evilfunhouse.com>
In "Definition: Sum” [1]: > In this way, Sum({1, 2, 3}) = op:numeric-add(1, op:numeric-add(2, op:numeric-add(3, 0))). I believe `Sum({1, 2, 3})` here indicates the application of `Sum` to the multiset `{1, 2, 3}`. However, aggregation is defined as operating over multisets *of lists*[2], so I think the argument to Sum here should be changed to either the multiset of lists `Sum({(1), (2), (3)})`, or the `Flatten`ed sequence `Sum((1, 2, 3))` (based on the definition of `Sum(S)` as opposed to `Sum(M)`). Similar issues exist with the examples in definitions for Avg, GroupConcat, and Sample. Also of note, there seems to be some stylistic variation in that the examples are included in the "Definition" div for Sum, but outside the div for Avg, GroupConcat, and Sample. Thanks, Greg [1] https://www.w3.org/TR/sparql11-query/#defn_aggSum <https://www.w3.org/TR/sparql11-query/#defn_aggSum> [2] https://www.w3.org/TR/sparql11-query/#defn_algAggregation <https://www.w3.org/TR/sparql11-query/#defn_algAggregation>
Received on Tuesday, 6 December 2022 21:22:20 UTC