Skip to content

[DO NOT MERGE] Copy of #2744 - #2762

Draft
Xiaoyu Z (xiaoyu-work) wants to merge 1 commit into
mainfrom
fix-docs-typos
Draft

Xiaoyu Z (xiaoyu-work) wants to merge 1 commit into
mainfrom
fix-docs-typos

Conversation

@xiaoyu-work

Copy link
Copy Markdown
Member

Describe your changes

Independent copy of #2744: Fix typos and a stale model reference in the docs.

Copies Iyanuoluwa007/Olive:fix-docs-typos to the same branch name in microsoft/Olive, preserving the original commits. The original PR remains unchanged.

Validation: the copied branch head matches the source PR head. Tests and lintrunner -a were not run for this copy-only operation; the existing code has not been revalidated.

Checklist before requesting a review

  • Add unit tests for this change.
  • Make sure all tests can pass.
  • Update documents if necessary.
  • Lint and apply fixes to your code by running lintrunner -a
  • Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

Source PR: #2744.

From the documentation items in the codebase review:

- README: "the the", "documentd".
- notebooks/README: "ocmmand", "hot wo".
- how-to index: the CLI command is capture-onnx-graph, not
  onnx-graph-capture; list optimize first instead of the deprecated
  auto-opt.
- getting started: the optimize example uses Qwen2.5-0.5B-Instruct, not
  Llama-3.2-1B-Instruct; fix the grammar and drop the fixed 60s estimate.
- Two more typos found with codespell ("atleast", "simlar").

Part of #2516

This branch has not been deployed

No deployments
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