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

Add Agg.medianAll(), Agg.percentileAll() to return a Seq of multiple values #221

Open
lukaseder opened this issue Mar 20, 2016 · 0 comments
Open

Add Agg.medianAll(), Agg.percentileAll() to return a Seq of multiple values #221

lukaseder opened this issue Mar 20, 2016 · 0 comments
Labels
P: Medium R: Fixed T: Enhancement
Milestone
Version 0.9.16

Comments

@lukaseder
Copy link
Member

See also #220
@lukaseder lukaseder added this to the Version 0.9.11 milestone Mar 20, 2016
TRebirthC added a commit to TRebirthC/jOOL that referenced this issue May 31, 2020
@TRebirthC
[jOOQ#221]Add Agg.medianAll() and Agg.percentileAll()
07eba36 
[jOOQ#221]Add Agg.medianAll() and Agg.percentileAll()
JingHuaMan added a commit to JingHuaMan/jOOL that referenced this issue May 16, 2021
@JingHuaMan
jOOQ#221 Add functions: percentileAllBy, medianAllBy
b83b357
JingHuaMan added a commit to JingHuaMan/jOOL that referenced this issue May 16, 2021
@JingHuaMan
jOOQ#221 Add tests for percentileAllBy, medianAllBy
a9467bd
@JingHuaMan JingHuaMan mentioned this issue May 20, 2021
Merged
lukaseder added a commit that referenced this issue May 21, 2021
@lukaseder
Merge pull request #385 from JingHuaMan/issue-221
2a232d9 
#221 Add percentileAll and medianAll
lukaseder added a commit that referenced this issue May 21, 2021
@lukaseder
[#221] Add Agg.medianAll(), Agg.percentileAll() to return a Seq of mu…
d41bd54 
…ltiple values

A few fixes from the suggested PR:

- Removed unnecessary comments
- Removed unnecessary covariance in Tuple2<T, U>
- Added static imports where appropriate
- Factored out common Collector logic between percentileBy() and percentileAllBy()
  - percentileCollector() is a Collector for common supplier/accumulator/combiner
    logic as well as handling empty and single element cases and sorting
  - percentileIndex does the percentile to arraylist index calculation
lukaseder added a commit that referenced this issue May 21, 2021
@lukaseder
[#221] Add Agg.medianAll(), Agg.percentileAll() to return a Seq of mu…
1bdd69b 
…ltiple values

A few fixes from the suggested PR:

- Removed unnecessary comments
- Removed unnecessary covariance in Tuple2<T, U>
- Added static imports where appropriate
- Factored out common Collector logic between percentileBy() and percentileAllBy()
  - percentileCollector() is a Collector for common supplier/accumulator/combiner
    logic as well as handling empty and single element cases and sorting
  - percentileIndex does the percentile to arraylist index calculation
lukaseder added a commit that referenced this issue May 21, 2021
@lukaseder
[#221] Removed more unnecessary comments
b69c5b7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
P: Medium R: Fixed T: Enhancement
Projects
None yet
Milestone
Version 0.9.16
Development

No branches or pull requests
1 participant
@lukaseder