Skip to content

agg is too complex for people and llms #1635

@dmpetrov

Description

@dmpetrov

Description

It's hard to comprehend that agg() should return iterator/list, not a single value per class. We need to change that.

I'd propose deprecating agg() via warning for now, introduce a new, value-returning function. And then rename the function to agg() and the current agg() to something like agg_n() or just remove it.

The value-returning aggregate function name could be acc() (accumulate), fus (fuse), dst (distil), red() (reduce), agg1() (silly), grp() (conflicts with group_by).

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions