Deduplicated multi-venue Position calc aggregation test suite #61

New Issue

goodboy · 2026-01-07T19:29:48Z

goodboy commented

2026-01-07 19:29:48 +00:00

Per the shell-test addedin f5850fe5 and landed in https://www.pikers.dev/pikers/piker/pulls/60 we need to actually fill out the suite with what’s missing,

use an example problematic ledger file with same-market-pair but diff clearing venues (likely from one of my ib.algopape accounts) to reproduce the case where the Position.fqme isn’t sufficient to map diff txns to the same pp.
ensure the test verifies txn-de-duplication / same-pp-calc-aggregation
emission of wtv warnings appropriate to the user?

Per the shell-test addedin f5850fe5 and landed in https://www.pikers.dev/pikers/piker/pulls/60 we need to actually fill out the suite with what's missing, - use an example problematic ledger file with same-market-pair but diff clearing venues (likely from one of my `ib.algopape` accounts) to reproduce the case where the `Position.fqme` isn't sufficient to map diff txns to the same pp. - ensure the test verifies txn-de-duplication / same-pp-calc-aggregation - emission of wtv warnings appropriate to the user?

goodboy changed title from ~~De-duplicated multi-venue `Position`s test suite~~ to Deduplicated multi-venue `Position` calc aggregation test suite

2026-01-07 19:30:12 +00:00

Sign in to join this conversation.

No Label

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: pikers/piker#61

Deduplicated multi-venue Position calc aggregation test suite #61

Deduplicated multi-venue `Position` calc aggregation test suite #61