Commit 4439015
committed
[issue-1430] feat: update aggregation functions for consistency and safety
The `minFor`, `medianFor`, and `percentileFor` functions were updated to use the `invoke` pattern for Aggregators, eliminating unchecked casts and enhancing consistency with `maxFor`. All changes passed tests without errors, and the solution has been submitted for review.1 parent 7772835 commit 4439015
File tree
3 files changed
+8
-40
lines changed- core/src/main/kotlin/org/jetbrains/kotlinx/dataframe/api
3 files changed
+8
-40
lines changedLines changed: 3 additions & 15 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
141 | 141 | | |
142 | 142 | | |
143 | 143 | | |
144 | | - | |
145 | | - | |
146 | | - | |
147 | | - | |
148 | | - | |
| 144 | + | |
149 | 145 | | |
150 | 146 | | |
151 | 147 | | |
| |||
336 | 332 | | |
337 | 333 | | |
338 | 334 | | |
339 | | - | |
340 | | - | |
341 | | - | |
342 | | - | |
343 | | - | |
| 335 | + | |
344 | 336 | | |
345 | 337 | | |
346 | 338 | | |
| |||
513 | 505 | | |
514 | 506 | | |
515 | 507 | | |
516 | | - | |
517 | | - | |
518 | | - | |
519 | | - | |
520 | | - | |
| 508 | + | |
521 | 509 | | |
522 | 510 | | |
523 | 511 | | |
| |||
Lines changed: 2 additions & 10 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
84 | 84 | | |
85 | 85 | | |
86 | 86 | | |
87 | | - | |
88 | | - | |
89 | | - | |
90 | | - | |
91 | | - | |
| 87 | + | |
92 | 88 | | |
93 | 89 | | |
94 | 90 | | |
| |||
219 | 215 | | |
220 | 216 | | |
221 | 217 | | |
222 | | - | |
223 | | - | |
224 | | - | |
225 | | - | |
226 | | - | |
| 218 | + | |
227 | 219 | | |
228 | 220 | | |
229 | 221 | | |
| |||
Lines changed: 3 additions & 15 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
156 | 156 | | |
157 | 157 | | |
158 | 158 | | |
159 | | - | |
160 | | - | |
161 | | - | |
162 | | - | |
163 | | - | |
| 159 | + | |
164 | 160 | | |
165 | 161 | | |
166 | 162 | | |
| |||
401 | 397 | | |
402 | 398 | | |
403 | 399 | | |
404 | | - | |
405 | | - | |
406 | | - | |
407 | | - | |
408 | | - | |
| 400 | + | |
409 | 401 | | |
410 | 402 | | |
411 | 403 | | |
| |||
616 | 608 | | |
617 | 609 | | |
618 | 610 | | |
619 | | - | |
620 | | - | |
621 | | - | |
622 | | - | |
623 | | - | |
| 611 | + | |
624 | 612 | | |
625 | 613 | | |
626 | 614 | | |
| |||
0 commit comments