[Docs] Follow TileOPs' op renames, and give the transform family a page - #63
Merged
Merged
Conversation
TileOPs #2410 renamed 27 manifest entries; the API pages, the manifest examples and this repository's CLAUDE.md named the old ones. Three renames came with #2401 and were never followed here, so the site does not build against today's main: `RopeNeoxFwdOp` and `RopeNonNeoxFwdOp` merged into `RopeFwdOp`, the three vector-norm ops merged into `VectorNormFwdOp`, and `DeltaNetAutogradFwdOp` was deleted. Each page the merge touched says in one line which parameter now picks what the separate ops used to. `docs/api/transform.md` is new, between FFT and mHC in the nav. `HadamardTransformFwdOp` is spec-only, so the page describes the signature `src/tileops/manifest/spec/transform.yaml` states rather than collecting a class mkdocs cannot import. Benchmarks put the transform family on the Other page: one op so far, and a rotation applied before quantization is not a quantization step. `check_api_pages.py` now reports 188 ops on the page and exported, with nothing the build cannot collect.
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.
Summary
CLAUDE.md.main:RopeNeoxFwdOpandRopeNonNeoxFwdOpmerged intoRopeFwdOp, the three vector-norm ops merged intoVectorNormFwdOp, andDeltaNetAutogradFwdOpwas deleted. Each merged entry's page says in one line which parameter now picks what the separate ops used to:ordfor the norm,rope_layoutfor the rotation.docs/api/transform.mdis new, between FFT and mHC in the nav.HadamardTransformFwdOpisspec-only, so the page describes the signaturesrc/tileops/manifest/spec/transform.yamlstates instead of collecting a class mkdocs cannot import.transformfamily on the Other page.