kucoin_and_binance_fix #9

Merged
goodboy merged 8 commits from kucoin_and_binance_fix into gitea_feats 2025-02-13 19:40:50 +00:00
Collaborator

Orig descr from @ntorres:

Added missing kucoinPair fields and deleted unnecessary field from binance FutesPair.


This PR is to land a small-ish set of piker.broker fixes namely for the binance and kucoin backends entailing,

  • various Pair field-schema adjustments to match the latest provider specs.
  • for kucoin repairing the live quote streaming which was borked after the move to using MktPair.bs_fqme: str as the dict-quote-msg primary key.
  • for binance also add a missing 'SPOT' venue-entry to the Client.pair2venuekey() key mapper..

NOTE, originally there were some disparate subsys changes that are now factored out into the dependent PR #29.

Orig descr from @ntorres: >Added missing kucoinPair fields and deleted unnecessary field from >binance FutesPair. --- This PR is to land a small-ish set of `piker.broker` fixes namely for the `binance` and `kucoin` backends entailing, - various `Pair` field-schema adjustments to match the latest provider specs. - for `kucoin` repairing the live quote streaming which was borked after the move to using `MktPair.bs_fqme: str` as the `dict`-quote-msg primary key. - for `binance` also add a missing `'SPOT'` venue-entry to the `Client.pair2venuekey()` key mapper.. --- NOTE, originally there were some disparate subsys changes that are now factored out into the dependent PR #29.
ntorres added 2 commits 2024-09-23 20:29:33 +00:00
goodboy added 2 commits 2024-10-09 20:11:21 +00:00
df34791d2f `kucoin`: repair live quotes streaming..
This must have broke at some point during the new `MktPair` and thus
`.fqme: str` updates; mas-o-menos the symbol key in the quote-msg-`dict`
was NOT set to the `MktPair.bs_fqme: str` value and thus wasn't being
processed by the downstream sampling and feed subsys.

So fix that as well as a few other refinements,
- set the `topic: mkt.bs_fqme` in quote msgs obvi.
- drop the "wait for first clearing vlm" quote poll loop; going to fix
  the sampler to handle a `first_quote` without a `'last'` key.
- add some typing around calls to `get_mkt_info()`.
- rename `stream_messages()` -> `iter_normed_quotes()`.
27df649fbf .clearing._ems: Don't require `first_quote['last']`
Instead just check for the field (which i'm not huge on the key-name for
anyway) and if not found get the "last price" from the real-time shm
buffer's latest 'close' sample.

Unrelatedly, use a `subs.copy()` in the `Router.client_broadcast()` loop
such that if a `client_stream` is popped on connection failure, we don't
RTE for the "size changed on iteration".
goodboy added 1 commit 2024-10-30 16:50:58 +00:00
41cd6ea852 Allow ledger passes to ignore (symcache) unknown fqmes
For example in the paper-eng, if you have a backend that doesn't fully
support a symcache (yet) it's handy to be able to ignore processing
other paper-eng txns when all you care about at the moment is the
simulated symbol.
goodboy force-pushed kucoin_and_binance_fix from 41cd6ea852 to e4d6f0940d 2024-10-30 17:28:06 +00:00 Compare
ntorres force-pushed kucoin_and_binance_fix from e4d6f0940d to 3e4e7c889b 2025-01-29 15:57:48 +00:00 Compare
ntorres force-pushed kucoin_and_binance_fix from 3e4e7c889b to 0223074d24 2025-01-29 18:42:10 +00:00 Compare
ntorres force-pushed kucoin_and_binance_fix from 0223074d24 to e4ed956eed 2025-01-30 18:09:50 +00:00 Compare
ntorres changed title from kucoin_and_binance_fix to kucoin_and_binance_fix 2025-01-30 18:48:17 +00:00
ntorres changed target branch from nix-qt6-fix to uv_migration 2025-01-30 18:48:17 +00:00
ntorres force-pushed kucoin_and_binance_fix from e4ed956eed to fe59ffc63f 2025-02-03 21:56:00 +00:00 Compare
ntorres force-pushed kucoin_and_binance_fix from fe59ffc63f to dd0d2bd07f 2025-02-03 22:08:50 +00:00 Compare
goodboy force-pushed kucoin_and_binance_fix from dd0d2bd07f to 7eacab0ac5 2025-02-11 23:29:46 +00:00 Compare
goodboy changed title from kucoin_and_binance_fix to kucoin_and_binance_fix 2025-02-11 23:30:28 +00:00
goodboy changed target branch from uv_migration to service_mng_to_tractor 2025-02-11 23:30:29 +00:00
goodboy force-pushed kucoin_and_binance_fix from 7eacab0ac5 to e69796ad6d 2025-02-11 23:43:22 +00:00 Compare
goodboy changed title from kucoin_and_binance_fix to kucoin_and_binance_fix 2025-02-11 23:48:55 +00:00
goodboy changed target branch from service_mng_to_tractor to go_httpx 2025-02-11 23:48:55 +00:00
goodboy force-pushed kucoin_and_binance_fix from e69796ad6d to 2a9fdec253 2025-02-12 13:02:50 +00:00 Compare
goodboy changed title from kucoin_and_binance_fix to kucoin_and_binance_fix 2025-02-12 13:04:43 +00:00
goodboy changed target branch from go_httpx to gitea_feats 2025-02-12 13:04:43 +00:00
goodboy force-pushed kucoin_and_binance_fix from 2a9fdec253 to 68940e8a73 2025-02-12 16:30:02 +00:00 Compare
goodboy force-pushed kucoin_and_binance_fix from 68940e8a73 to cb88dfc9da 2025-02-12 20:06:50 +00:00 Compare
goodboy added 2 commits 2025-02-12 23:08:57 +00:00
goodboy added 1 commit 2025-02-13 17:37:11 +00:00
goodboy added 1 commit 2025-02-13 18:38:43 +00:00
00108010c9 Mask `pytest` detection block in `piker.config`
Seems to be some kinda super weird env bug since we moved to using
`uv`? When it triggers it also seems to cause a pretty fundamental crash
that not only breaks `tractor.devx._debug` stuff but also seems to get
us in a perma-hang state where no SIGINT or other sys sig will be able
to kill the root proc!?!?

TODO, a `gitea` issue to track so we can fix the fundamental problem as
well as transitive fault in `tractor`'s core which seems to be due to
the error taking place during a sub-actor's module import phase which
prevents the runtime from booting fully and then the proc getting stuck
in a real gnarly SIG-state..
goodboy merged commit c933f2ad56 into gitea_feats 2025-02-13 19:40:50 +00:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 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#9
There is no content yet.