Implementation of distinctBy for DurationSumFilter/-Select#3839
Implementation of distinctBy for DurationSumFilter/-Select#3839awildturtok wants to merge 2 commits intodevelopfrom
Conversation
There was a problem hiding this comment.
können wir die final machen ?
There was a problem hiding this comment.
| private ColumnId column; | |
| private ColumnId dateRangeColumn; |
There was a problem hiding this comment.
Was passiert in dem Fall wenn distinctBy = []
There was a problem hiding this comment.
habs abgefangen, guter Punkt das ist natürlich quatsch
There was a problem hiding this comment.
Hier ist noch ein TODO und es ist abgeschnitten
There was a problem hiding this comment.
Ups, aber der Todo reisst es schon an, es hängt von einem anderen PR ab. Die aktuelle implementierung von DurationSum ist falsch.
| result,dates,concept select | ||
| 1,{2010-01-01/2010-01-01},1 | ||
| 3,{2010-01-01/2010-01-01}, | ||
| 4,{2010-01-01/2010-01-02},1 | ||
| 5,{2010-01-01/2010-01-02},2 | ||
| 6,{-∞/2010-01-01}, No newline at end of file |
There was a problem hiding this comment.
Wow das sieht irgendwie gefährlich und unintuitiv aus. Das Ergebnis hängt von der Sortierung der Zeilen ab 👀
There was a problem hiding this comment.
Naja, es geht am Ende nicht anders. Es geht ja darum, dass bestimmte Rezepte oä nur einmal gezählt werden. Aber ist definitiv wert in der Doku zu erwähnen, guter Punkt.
2 participants
No description provided.