pikers
/
piker
4
1
Fork 0
Code Issues 11 Pull Requests 13 Packages Projects Releases Wiki Activity

providers_sync: required API updates and .brokers refinements #56

Open
goodboy wants to merge 15 commits from providers_sync into main
pull from: providers_sync
merge into: pikers:main
Conversation 0 Commits 15 Files Changed 10
goodboy commented 2026-01-06 20:39:00 +00:00
Owner

Various required 2025 data/broker backend updates according to breaking changes namely from binance’s data APIs.

This patch is a re-factoring of related changes and replaces #40 with a follow up .brokers.ib change-set in suit as #IBPRNUMBERHERE.

Also included,

Various **required** 2025 data/broker backend updates according to breaking changes namely from `binance`'s data APIs. This patch is a re-factoring of related changes and replaces #40 with a follow up `.brokers.ib` change-set in suit as #IBPRNUMBERHERE. --- Also included, -
goodboy added 15 commits 2026-01-06 20:39:00 +00:00
9e7ad816d4 `.kraken`: add masked pauses for order req debug 
Such that the next time i inevitably must debug the some order-request
error status or precision discrepancy, i have the mkt-symbol branch
ready to go. Also, switch to `'action': 'buy'|'sell' as action,` style
`case` matching instead of the post-`if` predicate style.
a578f7a21f Update `binance` spot pairs with `amendAllowed` 
As per API updates,
https://developers.binance.com/docs/binance-spot-api-docs
https://developers.binance.com/docs/binance-spot-api-docs/faqs/order_amend_keep_priority

I also slightly tweaked the filed mismatch exception note to include the
`repr(pair_type)` so the dev can know which pair types should be
changed.
10a6f666b9 kraken: `Pair.costmin` is now optional? 
Some pairs don't seem to define it but it's not listed as deprecated on
official API page (new one now linked in type def's doc string).
d1ecbd8142 kraken: add crash-handling around `Pair()` init 
Since it can otherwise be difficult to debug due to nursery cancellation
(we need that taskman yo!).
16c73a0e5a binance: set `Pair.pegInstructionsAllowed = False` 
Lol, a cheeky unforeseen bug due to TOML's lack of a null type and
thinking i can render an `Optional` field on a `msgspec.Struct`
(defaulted to `None`) the `binance.symcache.toml` cache file..

I didn't catch this when i first updated to the 3.1 API in f7caa75228
because i never did a cache-files flush.. lesson learned and we **really
need tests for this**!!
254bb57ecc binance: add `Pair.opoAllowed` field 
Handle new API field per 2025-12-02 update.

(this patch was generated in some part by [`claude-code`][claude-code-gh])
[claude-code-gh]: https://github.com/anthropics/claude-code
This Pull Request doesn't have enough approvals yet. 0 of 1 approvals granted.
You are not authorized to merge this pull request.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
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#56
There is no content yet.
Delete Branch "providers_sync"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?