Skip to content

chore: bump pypto-lib for pl.KernelType syncall enums - #197

Merged
superxf merged 1 commit into
hw-native-sys:mainfrom
ndleslx:chore/bump-pypto-lib-kerneltype
Aug 26, 2026
Merged

chore: bump pypto-lib for pl.KernelType syncall enums#197
superxf merged 1 commit into
hw-native-sys:mainfrom
ndleslx:chore/bump-pypto-lib-kerneltype

Conversation

@ndleslx

@ndleslx ndleslx commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

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) and pl.SyncAllMode — and our CI resolves pypto from
HEAD (ci.yml resets the pypto cache to origin/HEAD). The pinned
pypto-lib (2cd5f828, before its #1017) still passes strings:

pypto.language.parser.diagnostics.exceptions.InvalidOperationError:
pl.system operation 'syncall': syncall core_type must be a KernelType
member, got 'mix'. Valid values: KernelType.AIC, KernelType.AIV, KernelType.MIX

raised while JIT-compiling qwen3_prefill_host
(pypto-lib/models/qwen3_14b/prefill_fwd.py), so the worker dies at
init_device_and_model and the engine never starts.

What

Bump the pypto-lib submodule 2cd5f828 → 149a5b8 (current origin/main
tip; first commit stops at the enum migration 6e5cd5d = pypto-lib #1017
for 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_resolve drop). Compatibility checks:

  • All sync-op call sites in the tree are enum-spelled from 6e5cd5d on
    (the remaining core_type="mixed" strings are on pl.jit.extern, an
    interface #2482 did not touch).
  • Nothing the range renames or drops (enable_l2_swimlane
    enable_chip_swimlane #1012, the MTP hc_pre syncall variant #1029,
    dspark decode output entries #1003, allow_early_resolve #1042) is
    referenced from pypto_serving/tests/scripts.
  • The modules serving imports by name (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

@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 68d6e4c1-b07b-4813-a0c2-9e8d3ea1e7e4


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ndleslx
ndleslx force-pushed the chore/bump-pypto-lib-kerneltype branch from b097b68 to 1885537 Compare August 26, 2026 01:51
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.
@superxf
superxf merged commit 80d24d5 into hw-native-sys:main Aug 26, 2026
3 of 4 checks passed
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