Skip to content

feat(collect): Collection.is_grouped / meta.grouped flag (#790) - #794

Merged
jepegit merged 5 commits into
masterfrom
790-collection-is-grouped
Jul 29, 2026
Merged

feat(collect): Collection.is_grouped / meta.grouped flag (#790)#794
jepegit merged 5 commits into
masterfrom
790-collection-is-grouped

Conversation

@jepegit

@jepegit jepegit commented Jul 29, 2026

Copy link
Copy Markdown
Owner

collect_summaries(group_it=True) silently returns a wide, non-averaged frame when a group has < 2 cells. This surfaces whether averaging actually happened via CollectionMeta.grouped + a Collection.is_grouped property, so apps no longer sniff for a mean column. Defaults False; round-trips through save/load.

Tests: grouped→True, single-cell fallback→False, default→False, save/load survives.

Closes #790

🤖 Generated with Claude Code

collect_summaries(group_it=True) silently falls back to a wide, non-averaged
frame when a group has < 2 cells. Surface whether averaging actually happened:
CollectionMeta.grouped (set by collect_summaries) + a Collection.is_grouped
property, so apps stop sniffing for a "mean" column. Field defaults False and
round-trips through save/load.

Closes #790

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@jepegit
jepegit enabled auto-merge (squash) July 29, 2026 11:27
@cursor

cursor Bot commented Jul 29, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@jepegit
jepegit merged commit 9c9e6e9 into master Jul 29, 2026
5 checks passed
@jepegit
jepegit deleted the 790-collection-is-grouped branch July 29, 2026 12:50
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.

Signal whether group-averaging happened (Collection.is_grouped / meta.grouped)

1 participant