pikers
/
piker
3
0
Fork 0
Code Issues 8 Pull Requests 10 Packages Projects Releases Wiki Activity

Nixos default.nix file for Qt6 #7

Closed
ntorres wants to merge 25 commits from nix-qt6-fix into go_httpx
pull from: nix-qt6-fix
merge into: pikers:go_httpx
Conversation 3 Commits 25 Files Changed 36
ntorres commented 2024-08-15 00:10:41 +00:00
Collaborator

Default.nix file adapted to use Qt6, pyproject.toml file modify, now cython and greenback are under dev group.

Default.nix file adapted to use Qt6, pyproject.toml file modify, now cython and greenback are under dev group.
ntorres added 4 commits 2024-08-15 00:10:41 +00:00
5ec756234a Mask no-data pause-point, add perps to sig. 
Was orig for debugging an issue with `kucoin` i think but definitely
shouldn't be left in XD

Also add `'perpetual_future'` to `start_backfill()` input literal set.
guille approved these changes 2024-08-15 15:07:42 +00:00
guille left a comment
Collaborator

can confirm works

can confirm works
sc.png
211 KiB
sc.png
ntorres added 2 commits 2024-08-23 17:37:01 +00:00
cc5b21a7e6 Prep service mngr for move to `tractor.hilevel` 
Given it's a fairly simple yet useful abstraction, it makes sense to
offer this sub-sys alongside the core `tractor` runtime lib.

Without going into extreme detail on the impl changes (it'll come in
the commit that moves to the other repo) here is the high level summary:
------ - ------
- rename `Services` -> `ServiceMngr` and use an factory `@acm`
  to guarantee a single-instance-per-actor using a niche approach for a
  singleton object using a default keyword-arg B)
  - the mod level `open_service_mngr()` and `get_service_mngr()` are the
    new allocation/access API.
- add a `ServiceMngr.start_service()` method which does the work of both
  spawning a new subactor (for the daemon) and uses its portal to start
  the mngr side supervision task.
- open actor/task nurseries inside the `@acm` allocator.

Adjust other dependent subsystems to match:
------ - ------
- use `open_service_mngr()` when first allocated in `open_pikerd()`.
- use `get_service_mngr()` instead of importing the class ref inside
  `.service.maybe_spawn_daemon()`, `.brokers._daemon.spawn_brokerd()`
  and `.data._sampling.spawn_samplerd()` using a `partial` to pack in
  the endpoint ctx kwargs (unpacked inside `.start_service()` XD).
de6189da4d Drop `.cancel_actor()` from `maybe_spawn_daemon()` 
Since `tractor`'s new and improved inter-actor cancellation semantics
are much more pedantic, AND bc we use the `ServiceMngr` for spawning
service actors on-demand, the caller of `maybe_spawn_daemon()` should
NEVER conduct a so called "out of band" `Actor`-runtime cancel request
since this is precisely the job of our `ServiceMngr` XD

Add a super in depth note explaining the underlying issue and adding
a todo list of how we should prolly augment `tractor` to make such cases
easier to grok and fix in the future!
ntorres added 1 commit 2024-08-23 18:08:05 +00:00
goodboy commented 2024-08-27 02:30:23 +00:00
Owner

So this is ready to land @NT?

So this is ready to land @NT?
👍 1
goodboy commented 2024-08-27 02:31:05 +00:00
Owner

can confirm works

lol bro you gotta re-config ur font size

> can confirm works lol bro you gotta re-config ur font size
ntorres commented 2024-08-29 21:28:34 +00:00
Poster
Collaborator

So this is ready to land @NT?

Yeah!!

> So this is ready to land @NT? Yeah!!
ntorres force-pushed nix-qt6-fix from 0be454c3d6 to 820bac2707 2025-01-29 03:06:12 +00:00 Compare
ntorres force-pushed nix-qt6-fix from 820bac2707 to 58f91990a4 2025-01-29 03:19:26 +00:00 Compare
ntorres force-pushed nix-qt6-fix from 58f91990a4 to ec2203168a 2025-01-29 03:35:04 +00:00 Compare
ntorres added 5 commits 2025-01-29 18:23:33 +00:00
d512037ed4 Added missing fields for kucoin. 
feeCategory, makerFeeCoefficient, takerFeeCoefficient and st.
3ebbfcb9ca `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()`.
afa5655af7 .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".
3e4e7c889b 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.

NOTE, that currently this will still result in a key-error when you load
more then one mkt with the paper engine (for which the backend does not
have the symcache implemented) since no fqme ad-hoc query was made for
the 2nd symbol (and i'm not sure we should support that kinda hackery
over just encouraging the sym-cache being added?). Def needs a little
more thought depending on how many backends are never going to be able
to (easily) support caching..
ntorres force-pushed nix-qt6-fix from 3e4e7c889b to bda7e69ad9 2025-01-29 18:41:25 +00:00 Compare
ntorres added 1 commit 2025-01-29 22:23:10 +00:00
ntorres closed this pull request 2025-01-30 18:46:39 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
guille
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
3 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#7
There is no content yet.
Delete Branch "nix-qt6-fix"

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?