Mark vendored and byte-gated trees; document the ops/ taxonomy (#78 phase 0) - #80
Draft
qywu wants to merge 1 commit into
Draft
Mark vendored and byte-gated trees; document the ops/ taxonomy (#78 phase 0)#80qywu wants to merge 1 commit into
qywu wants to merge 1 commit into
Conversation
…hase 0) No code moves — this is the mark-don't-move phase of #78: - Extend the pre-commit global exclude (which already covered quack/) to flashqla/ and bi_families_v2.py, so no rewriting hook can touch them. bi_families_v2.py was previously excluded only from ruff-format; the ruff hook runs --fix-only and could have silently rewritten the sha256-gated bytes on a rule upgrade. - Add the same excludes to [tool.ruff] in pyproject with force-exclude, so direct ruff invocations (which receive explicit file paths and ignore ordinary excludes) skip them too. - Skip the vendored trees in codespell. - Add src/xorl/ops/quack/VENDORED.md (provenance: Dao-AILab QuACK 0.4.1 partial fork over the quack-kernels PyPI pin; local-patch ledger: cute_dsl_elf_fix, cute_dsl_mlir_threading) generalizing the existing flashqla VENDORED.md pattern. - Add src/xorl/ops/README.md: the edit-policy table (vendored / byte-contract-gated / first-party), the current map of every subtree, and where each is headed under #78.
Broly Security ScanNote ✅ Clean scan Note Re-scan this PR anytime with
|
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.
Phase 0 of #78 — mark, don't move. Zero import churn; no Python file is modified.
exclude(already coveringquack/) now also coverslinear_attention/flashqla/andbi_families_v2.py. The latter was only excluded fromruff-formatbefore — but theruffhook runs--fix-only(a rewriter), so a rule upgrade could have silently edited the sha256-gated bytes.[tool.ruff]withforce-exclude = true, so ruff skips these paths even when they're passed explicitly (this is exactly how a tree-wideruff --fixrecently rewrote 45 vendored quack files).ops/quack/VENDORED.md: provenance (Dao-AILab QuACK, snapshotted at 0.4.1 as a partial fork that still imports helpers from thequack-kernels==0.5.0PyPI pin) and the local-patch ledger (cute_dsl_elf_fix,cute_dsl_mlir_threading), generalizing the existing flashqla pattern.ops/README.md: the edit-policy table (vendored / byte-contract-gated / first-party), a map of every subtree, and where each is headed under Reorganize src/xorl/ops: quarantine vendored code, separate kernels / layers / objectives, one home for the exact-contract family #78.Verified:
pre-commit run --all-files(the lint CI command) passes;ruff checkon the excluded paths now reports no files.Next: phase 1 (physically quarantine
quack/+flashqla/underops/_vendored/with re-export stubs) after #77 lands.