Skip to content

Fix #565: Review Time_Agg in group by / group except#591

Draft
mla2001 wants to merge 3 commits intomainfrom
cr-565
Draft

Fix #565: Review Time_Agg in group by / group except#591
mla2001 wants to merge 3 commits intomainfrom
cr-565

Conversation

@mla2001
Copy link
Contributor

@mla2001 mla2001 commented Mar 12, 2026

Summary

Reviewed and implemented Time_Agg in group by / group except

Checklist

  • Code quality checks pass (ruff format, ruff check, mypy)
  • Tests pass (pytest)
  • Documentation updated (if applicable)

@mla2001 mla2001 self-assigned this Mar 12, 2026
@mla2001 mla2001 requested review from a team and javihern98 March 12, 2026 11:34
@mla2001 mla2001 linked an issue Mar 12, 2026 that may be closed by this pull request
@mla2001 mla2001 changed the title Cr 565 Implemented #565: Review Time_Agg in group by / group except Mar 12, 2026
@mla2001 mla2001 added this to the 1.7.0 milestone Mar 12, 2026
@mla2001 mla2001 marked this pull request as draft March 12, 2026 11:40
@javihern98 javihern98 changed the title Implemented #565: Review Time_Agg in group by / group except Fix #565: Review Time_Agg in group by / group except Mar 12, 2026
@javihern98 javihern98 removed this from the 1.6.1 milestone Mar 16, 2026
@javihern98
Copy link
Contributor

@mla2001 We could set this one as ready and include it as well, in the end is also part of the standard 2.1...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Review Time_Agg in group by / group except

2 participants