Skip to content
This repository has been archived by the owner on May 10, 2022. It is now read-only.

Prototype of an aggs parser handle. #8

Open
wants to merge 25 commits into
base: master
Choose a base branch
from

Conversation

statwonk
Copy link
Contributor

This is more of a proposal than anything and I don't intend for a merge into master, at least not withstanding major polishing. Here's my little toy code for playing with aggs parsers. It's not recursive, which would be nice to have. @sckott as you'll see there's a departure from your structure. I'm a bit of a newcomer to lazy evaluation, so I'm sure that I'm missing important features. Comments are appreciated!

@statwonk
Copy link
Contributor Author

I think it might be nice to shoot for a syntax like,
aggs(my_statistic = mean(decimalLatitude))
aggs(my_statistics = quantile(decimalLatitude))
aggs(my_terms = terms(play_name))

The challenge comes in with nesting ...

@sckott
Copy link
Contributor

sckott commented Feb 29, 2016

thanks! I'll have a look in the morning

@statwonk
Copy link
Contributor Author

statwonk commented Mar 1, 2016

for my own reference: http://adv-r.had.co.nz/dsl.html

@sckott
Copy link
Contributor

sckott commented Mar 1, 2016

Probably is good, But let me get the #6 sorted out first, then it'll be easier to play with this

@statwonk
Copy link
Contributor Author

statwonk commented Mar 2, 2016

👍 One thing of mention re: last call, I think a verbose option that would allow the end user see the call construction in json could really help usability.

@sckott
Copy link
Contributor

sckott commented Mar 2, 2016

yeah, see #12

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants