knowsys / knowsys/nemo

Support more aggregates

Open
#782 0 comments 0 reactions 0 assignees View on GitHub
aggregates
Dominant language
Rust
Stars
289
Forks
21
PR merge metrics
No merged PRs in 30d

Description

Issue #642 contained a list of unimplemented builtin functions but also aggregates. Since they are handled quite differently in the implementation, I put them in a separate issue here. The missing aggregates are:

- `#avg`
- `#GroupConcat`
- `#Sample`

See https://www.w3.org/TR/sparql11-query/#aggregateAlgebra

Contributor guide

Open the contributing guide

Research direction

Start with the linked SPARQL 1.1 aggregate algebra specification, then locate the existing aggregate handling in the Rust code. Confirm the expected behavior for #avg, #GroupConcat, and #Sample and identify the relevant test entry points; done means all three aggregates are supported and covered by tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.