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

Jump to bottom

Fix aggregations and extend set of built-in functions #22

Open
8 of 11 tasks
comnik opened this issue Oct 9, 2018 · 1 comment
Open
8 of 11 tasks

Fix aggregations and extend set of built-in functions #22

comnik opened this issue Oct 9, 2018 · 1 comment
Assignees
@comnik @bachdavi

Comments

@comnik
Copy link
Owner

comnik commented Oct 9, 2018 •
edited by bachdavi
Loading

  • COUNT is actually SUM at the moment, because I am an idiot
  • MIN and MAX do not group correctly
  • No support for something like Datomic's:with clause
  • Potentially hairy type conversions
  • Aggregation might re-order relations, violating any underlying projection
  • COUNT-DISTINCT
  • AVG
  • MEDIAN
  • VARIANCE
  • STDDEV
  • Floating points

Datomic also offers a couple of things returning collections, some of which (e.g. distinct) directly map to Differential operators and should be treated as such.
@comnik
Copy link
Owner Author

comnik commented Oct 20, 2018

Floating points are experimental, via Value::Rational32, but seem to work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@comnik comnik

@bachdavi bachdavi

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@comnik @bachdavi