Skip to content

docs: remove duplicated words and garbled fragments in comments and docs - #8599

Open
simpleqt wants to merge 1 commit into
deepspeedai:masterfrom
simpleqt:sq919/dup-words
Open

simpleqt wants to merge 1 commit into
deepspeedai:masterfrom
simpleqt:sq919/dup-words

Conversation

@simpleqt

Copy link
Copy Markdown

A sweep of duplicated/garbled words, each verified verbatim before editing:

  • deepspeed/inference/v2/scheduling_utils.py (public enum docstrings) — 'would would overflow' and 'in a single forwardp' → 'would overflow', 'forward pass'
  • deepspeed/runtime/zero/partitioned_param_coordinator.py — TODO comment 'throw if if there are still active submodules'
  • deepspeed/runtime/fp16/loss_scaler.py — duplicated 'the' before dynamic_loss_args
  • docs/_tutorials/accelerator-abstraction-interface.md — 'is used get get communication backend name' → 'is used to get the communication backend name'
  • blogs/deepspeed-chat/ds-chat-release-8-31/README.md — 'in the the ZeRO GatheredParameters context'

Comments/docstrings/docs only; no code change.

Signed-off-by: simpleqt 89645338+simpleqt@users.noreply.github.com

- scheduling_utils enum docstrings: 'would would overflow', 'in a single
  forwardp' -> 'forward pass'
- partitioned_param_coordinator TODO: 'if if'
- loss_scaler comment: duplicated 'the' before dynamic_loss_args
- accelerator-abstraction tutorial: 'is used get get' -> 'is used to get the'
- ds-chat release blog: 'in the the ZeRO'

Signed-off-by: simpleqt <89645338+simpleqt@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings September 19, 2026 08:19

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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