appleby
appleby
Note: the tests have disappeared in the latest diff after rebasing onto master because they were identical with the tests added in https://github.com/metabase/metabase/pull/52692, which has now merged.
> I remember that I asked why we require both aggregation and breakout, and the answer was that it was by design. Link to previous discussion: https://github.com/metabase/metabase/pull/46670#discussion_r1724048435
> I had to check if `[-180, 180]` is actually canonical. AFAICT that is the case. There are some shenanigans with the International Date Line and certain time zones, see...
@noahmoss Not sure if it matters, but heads up you have both the `backport` and `no-backport` labels on this PR
> > 2. Visualize, notice rows count is 4308 (⚠ shouldn't it be limited to 2000?) > > I think that's the actual bug but I'm not entirely sure. Please...