chore: bump pypto-lib for pl.KernelType syncall enums - #197
Merged
superxf merged 1 commit intoAug 26, 2026
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ndleslx
force-pushed
the
chore/bump-pypto-lib-kerneltype
branch
from
August 26, 2026 01:51
b097b68 to
1885537
Compare
pypto #2482 typed the sync-op keywords as enums (pl.KernelType / pl.SyncAllMode) and CI resolves pypto from HEAD, so every guard that JIT-compiles a pypto-lib kernel with a string core_type died with TypeError: syncall core_type must be a KernelType member, got 'mix' pypto-lib migrated in #1017 (6e5cd5d); bump the pin past that to the current tip. Nothing renamed or dropped along the way (enable_l2_swimlane, the MTP hc_pre syncall variant, dspark decode output entries, allow_early_resolve) has a caller in this repo, and the modules serving imports by name change no public signature. The range also carries the Qwen3-14B prefill retune (#912) and the DeepSeek-V4 fixes up to #1045.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
CI is red on every branch (seen on #196's Qwen3 accuracy guard, run
https://github.com/hw-native-sys/pypto-serving/actions/runs/32845551402):
pypto #2482 typed the sync-op keywords as enums —
pl.KernelType(
AIC/AIV/MIX) andpl.SyncAllMode— and our CI resolves pypto fromHEAD (
ci.ymlresets the pypto cache toorigin/HEAD). The pinnedpypto-lib (
2cd5f828, before its #1017) still passes strings:raised while JIT-compiling
qwen3_prefill_host(
pypto-lib/models/qwen3_14b/prefill_fwd.py), so the worker dies atinit_device_and_modeland the engine never starts.What
Bump the
pypto-libsubmodule2cd5f828 → 149a5b8(current origin/maintip; first commit stops at the enum migration
6e5cd5d= pypto-lib #1017for easy bisection, second takes the tip). The five commits past the
migration are two CI-only changes and three DeepSeek-V4 kernel fixes
(#1019 Flash E2E restore, #1036 V4 attention decode layers, #1042
allow_early_resolvedrop). Compatibility checks:6e5cd5don(the remaining
core_type="mixed"strings are onpl.jit.extern, aninterface #2482 did not touch).
enable_l2_swimlane→enable_chip_swimlane#1012, the MTPhc_presyncall variant #1029,dspark decode output entries #1003,
allow_early_resolve#1042) isreferenced from
pypto_serving/tests/scripts.prefill_layer,prefill_fwd,prefill_mtp) change no public signature in the range.The range also carries the Qwen3-14B prefill tiling retune (#912) and the
DeepSeek-V4/dspark work up to the tip — pypto-lib's own CI validated
those; the guards below re-validate them behind serving.
Note for local runs: a local pypto older than #2482 (no
pl.KernelType)will now fail to compile these kernels — update the local pypto checkout.
🤖 Generated with Claude Code