You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
FWIW, here are the post 16.0 PRs I'm currently backporting for VegaFusion (jonmmease#126). It's basically #4873 and a handful of PRs to finish wiring things up so that mean and variance-based aggregates work in windows with custom frames.
@jonmmease I reviewed those PRs, and it seems reasonable to apply them all to the patch release. Could you create the necessary PR(s) to apply those changes to the maint-16.x branch?
Activity
alamb commentedon Jan 17, 2023
cc @jonmmease as I think you you may have some ideas of what would be good to backport too
Also cc @maxburke who I think was working with an upgrade
jonmmease commentedon Jan 17, 2023
FWIW, here are the post 16.0 PRs I'm currently backporting for VegaFusion (jonmmease#126). It's basically #4873 and a handful of PRs to finish wiring things up so that mean and variance-based aggregates work in windows with custom frames.
Thanks for the ping!
andygrove commentedon Jan 17, 2023
@jonmmease I reviewed those PRs, and it seems reasonable to apply them all to the patch release. Could you create the necessary PR(s) to apply those changes to the
maint-16.xbranch?jonmmease commentedon Jan 18, 2023
Sure thing
jonmmease commentedon Jan 18, 2023
@andygrove, turns out the window aggregated fixes did make it into 16.0.0, so there's only one backport PR needed for #4873: Here's the PR: #4959