Skip to content

Use keyword-only arguments in map_rows #1176

@hombit

Description

@hombit

Bug report

When porting code from reduce to map_rows, it is easy to forget to move column names from positional arguments to columns. Making all arguments except the first keyword-only, would help prevent duplicate keyword argument values, like in this example:

catalog.map_rows(func, "ra", "dec", row_container="args")
TypeError: Catalog.map_rows() got multiple values for argument 'row_container'

Environment Information

Before submitting
Please check the following:

  • I have described the situation in which the bug arose, including what code was executed, and any applicable data others will need to reproduce the problem.
  • I have included information about my environment, including the version of this package (e.g. lsdb.__version__)
  • I have included available evidence of the unexpected behavior (including error messages, screenshots, and/or plots) as well as a description of what I expected instead.
  • If I have a solution in mind, I have provided an explanation and/or pseudocode and/or task list.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingnested-pandasSupporting nested-pandas in LSDB

    Type

    No type

    Projects

    Status

    No status

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions