Skip to content

Update website documentation for groupBy and pivot#1800

Merged
Allex-Nik merged 6 commits into
masterfrom
groupBy-pivot-website-docs
May 12, 2026
Merged

Update website documentation for groupBy and pivot#1800
Allex-Nik merged 6 commits into
masterfrom
groupBy-pivot-website-docs

Conversation

@Allex-Nik
Copy link
Copy Markdown
Collaborator

@Allex-Nik Allex-Nik commented Apr 14, 2026

Fixes #1535

@Allex-Nik Allex-Nik added the documentation Improvements or additions to documentation (not KDocs) label Apr 15, 2026
@Allex-Nik Allex-Nik force-pushed the groupBy-pivot-website-docs branch from 2fb8981 to a607889 Compare April 21, 2026 13:46
@Allex-Nik Allex-Nik changed the title Update website documentation for groupBy Update website documentation for groupBy and pivot Apr 21, 2026
@Allex-Nik Allex-Nik force-pushed the groupBy-pivot-website-docs branch from a607889 to 729244f Compare April 21, 2026 15:39
@Allex-Nik Allex-Nik marked this pull request as ready for review April 21, 2026 16:22
@Allex-Nik Allex-Nik added this to the 1.0.0-Beta6 milestone Apr 21, 2026
Copy link
Copy Markdown
Collaborator

@Jolanrensen Jolanrensen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great job!

Comment thread core/src/test/kotlin/org/jetbrains/kotlinx/dataframe/samples/api/Analyze.kt Outdated
Comment thread docs/StardustDocs/topics/groupBy.md Outdated
Comment thread docs/StardustDocs/topics/groupBy.md Outdated
Comment thread docs/StardustDocs/topics/groupBy.md Outdated
Comment thread docs/StardustDocs/topics/groupBy.md Outdated

<!---FUN groupByDirectAggregations-->
* [`medianBy`](median.md) / [`percentileBy`](percentile.md) – to get the row with the median or specific percentile value of the given `RowExpression` calculated on rows within each group.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the row closest to the estimated median/percentile index of the results of the RowExpressions to be exact XD Maybe there's a better way to phrase it ;P but I don't want to mislead either. Of course they can always get more information by reading the median/percentile docs

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

they use QuantileEstimationMethod R3 to select the right row if you are curious :)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it fine now? Or is it better to formulate it differently?

Comment thread docs/StardustDocs/topics/groupBy.md Outdated
Comment thread docs/StardustDocs/topics/groupBy.md
Comment thread docs/StardustDocs/topics/groupBy.md Outdated
Comment thread docs/StardustDocs/topics/pivot.md Outdated
Comment thread docs/StardustDocs/topics/pivot.md Outdated
@Allex-Nik Allex-Nik force-pushed the groupBy-pivot-website-docs branch from 729244f to ee2c28b Compare April 28, 2026 14:19
@Allex-Nik Allex-Nik requested a review from Jolanrensen April 28, 2026 19:29
@Allex-Nik Allex-Nik modified the milestones: 1.0.0-Beta6, Post-RC May 4, 2026
Copy link
Copy Markdown
Collaborator

@AndreiKingsley AndreiKingsley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!
Requested some small changes, you can merge after fixing them.

Comment thread docs/StardustDocs/topics/groupBy.md Outdated
Comment thread docs/StardustDocs/topics/groupBy.md Outdated
Comment thread docs/StardustDocs/topics/groupBy.md Outdated
Comment thread docs/StardustDocs/topics/groupBy.md Outdated
Comment thread docs/StardustDocs/topics/groupBy.md Outdated
Comment thread docs/StardustDocs/topics/groupBy.md Outdated
Comment thread docs/StardustDocs/topics/groupBy.md Outdated
Copy link
Copy Markdown
Collaborator

@zaleslaw zaleslaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool! Can't wait till site redeploy to see there

@Allex-Nik Allex-Nik force-pushed the groupBy-pivot-website-docs branch from ee2c28b to b7b8e5a Compare May 8, 2026 13:56
@Allex-Nik Allex-Nik requested a review from AndreiKingsley May 8, 2026 15:15
Copy link
Copy Markdown
Collaborator

@AndreiKingsley AndreiKingsley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

@Allex-Nik Allex-Nik merged commit a6f2ceb into master May 12, 2026
5 checks passed
@Allex-Nik Allex-Nik deleted the groupBy-pivot-website-docs branch May 12, 2026 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation (not KDocs)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update groupBy and pivot website documentation

4 participants