-
-
Notifications
You must be signed in to change notification settings - Fork 336
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.
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
First letter grouping #1314
First letter grouping #1314
Conversation
This can be reviewed and merged as soon as #1312 is merged. Leaving as a draft until that point to avoid mistakes. |
Codecov Report
@@ Coverage Diff @@
## master #1314 +/- ##
==========================================
+ Coverage 93.49% 93.58% +0.09%
==========================================
Files 114 114
Lines 4426 4491 +65
==========================================
+ Hits 4138 4203 +65
Misses 288 288
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Fixes #408
This adds a grouping mode to the API which allows one to group by the first
n
characters of each entry in a column (after casting toTEXT
if needed).Technical details
n
is given by theprefix_length
parameter. Only works on single columns. In particular, this sorts out #408 for textual columns by giving"prefix_length": 1
.Also adds all
GroupBy
object properties to the returnedgrouping
JSON object.NOTE: This should be merged after #1312 .
Checklist
Update index.md
).master
branch of the repositoryvisible errors.
Developer Certificate of Origin
Developer Certificate of Origin