google/temporian

New operator: cumprod

Opened this issue · 3 comments

ianspektor commented

New EventSet.cumprod() operator.

Depends on #380.

See EventSet.cumsum() for reference.

See https://github.com/google/temporian/blob/main/CONTRIBUTING.md#developing-a-new-operator for guidance.

Questions or requests for additional guidance from possible contributors more than welcome!

akshatvishu commented

Hi @ianspektor! I'm happy to contribute to this issue.

ianspektor commented

Hey @akshatvishu!

When creating this issue I thought moving_product already existed, which isn't the case, which is why in the description I specified it should be implemented like cumsum is, which uses moving_sum.

Creating new ticket for that first operator, that this one depends on: #380

akshatvishu commented

Hey @akshatvishu!

When creating this issue I thought moving_product already existed, which isn't the case, which is why in the description I specified it should be implemented like cumsum is, which uses moving_sum.

Creating new ticket for that first operator, that this one depends on: #380

I think I should do both of them together!