Refactor: replace deprecated pl.incore/pl.auto_incore with pl.at syntax#101
Conversation
Migrate all usages of `pl.incore()` and `pl.auto_incore()` to the new unified `pl.at(level=pl.Level.CORE_GROUP)` API across all examples and docs. Also remove the obsolete scope12 file.
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThis PR systematically replaces Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related issues
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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 |
There was a problem hiding this comment.
Code Review
This pull request updates the PyPTO API by replacing deprecated pl.auto_incore() and pl.incore() calls with the new pl.at() scope across documentation and model examples. The transition to with pl.at(level=pl.Level.CORE_GROUP, ...) provides more explicit control over execution levels and enables optimizations like the chunked loop optimizer. These changes are reflected in core documentation and a wide range of model implementations, including DeepSeek v3.2, Kimi K2, MiLM, and Qwen3-32B. Additionally, a redundant Qwen3 decode script was removed. I have no feedback to provide as there are no review comments.
Summary
pl.incore()withpl.at(level=pl.Level.CORE_GROUP)across examples and docspl.auto_incore()withpl.at(level=pl.Level.CORE_GROUP, optimization=pl.chunked_loop_optimizer)across examples and docsqwen3_32b_decode_scope12.pyRelated Issues
N/A