Skip to content

enc_hook & dec_hook factory (+ CI uv update)#376

Draft
guilledk wants to merge 4 commits intomainfrom
auto_codecs
Draft

enc_hook & dec_hook factory (+ CI uv update)#376
guilledk wants to merge 4 commits intomainfrom
auto_codecs

Conversation

@guilledk
Copy link
Collaborator

No description provided.

box_types: dict[Type, Struct] = {}
for ext_type in ext_types:
info = msgspec.inspect.type_info(ext_type)
if isinstance(info, CustomType):
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

docs on all this in at least the doc-string would be great ;)

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also obvi linking the msgpec docs where possible as well 🙏🏼

# yield ext_codec
#
# ^-TODO-^ is it impossible to make something like this orr!?
# builtins we can have in same pld_spec as custom types
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

says who? link to msgspec docs pretty please :)

goodboy added a commit that referenced this pull request Apr 26, 2025
Shared memory array API and optional tight integration with `numpy`

Landing this so many downstream major feature branches depend on it namely,
- #375
- #376
- and the eventual https://pikers.dev/goodboy/tractor/pulls/10
@goodboy goodboy changed the title CI uv update + enc_hook & dec_hook factory enc_hook & dec_hook factory (+ CI uv update) Jun 21, 2025
@goodboy goodboy force-pushed the structural_dynamics_of_flow branch from fbc9325 to 27e6ad1 Compare July 11, 2025 02:05
Base automatically changed from structural_dynamics_of_flow to main July 13, 2025 19:11
@goodboy
Copy link
Owner

goodboy commented Jul 13, 2025

@guilledk trying to figure out what we can actually distill from this without bringing in too much more then delegating to msgspec.

Also this needs a rebase!

@goodboy
Copy link
Owner

goodboy commented Aug 20, 2025

@guilledk needs a big rebase, but also this ended up getting moved elsewhere yah?

@goodboy
Copy link
Owner

goodboy commented Sep 25, 2025

Gotta rebase this one obvi but i think despite most of this stuff in a more refined form (i think) getting moved elsewhere, there's still a couple things we might want to keep. So i will be going through it, maybe updating and landing some content.

@goodboy goodboy marked this pull request as draft September 25, 2025 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants