-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Feature/5820 dataframe extend groupby #5821
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
Feature/5820 dataframe extend groupby #5821
Conversation
Merge latest changes from dotnet/machinelearning
Back merge from base repository
Codecov Report
@@ Coverage Diff @@
## main #5821 +/- ##
==========================================
+ Coverage 68.32% 68.35% +0.02%
==========================================
Files 1131 1132 +1
Lines 241291 241368 +77
Branches 25053 25059 +6
==========================================
+ Hits 164863 164978 +115
+ Misses 69923 69888 -35
+ Partials 6505 6502 -3
Flags with carried forward coverage won't be shown. Click here to find out more.
|
|
I pushed a commit to fix the remaining comments, so we can be ready for the next preview release. Great work here @asmirnov82, thanks! |
eerhardt
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
![]()
Fixes #5820
Extend DataFrame GroupBy operation by adding new property Groupings. This property returns collection of IGrouping objects (the same way as LINQ GroupBy operation does). This allows to use syntacs like this:
var groups = dataFrame.GroupBy<TKey>(columnName).Groupings.ToDictionary(g => g.Key, g => g.ToList()); foreach (DataFrameRow row in groups[KeyValue]) { // any code to work with row object }