pola-rs/polars

.list.product

Open

#12,862 opened on Dec 2, 2023

View on GitHub
 (6 comments) (4 reactions) (1 assignee)Rust (38,496 stars) (2,826 forks)batch import
acceptedenhancementgood first issue

Description

Description

If there's .list.sum, perhaps .list.product would also be in-scope?

Current workaround

In [30]: s
Out[30]:
shape: (2,)
Series: '' [list[i64]]
[
        [1, 2, 3]
        [4, 5, 6]
]

In [31]: s.list.eval(pl.element().product()).list.first()
Out[31]:
shape: (2,)
Series: '' [i64]
[
        6
        120
]

Contributor guide

Tech stack
pythonrust
Domain
data
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
blocked
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Polars Python API basicsRust basicsList comprehension
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60
Research direction
Investigate the existing implementation of `.list.sum` in the Polars codebase. The Python side likely defines the method in `polars/series/list.py`, and the actual computation is implemented in Rust in the list namespace. Follow the pattern to add a `.list.product` method, ensuring it mirrors the sum implementation but uses multiplication. Check if there are any type considerations or edge cases.