Skip to content

Fix typos in comments, docstrings, and error messages#1239

Open
maxtaran2010 wants to merge 1 commit into
python-control:mainfrom
maxtaran2010:fix/typos-in-comments-and-docstrings
Open

Fix typos in comments, docstrings, and error messages#1239
maxtaran2010 wants to merge 1 commit into
python-control:mainfrom
maxtaran2010:fix/typos-in-comments-and-docstrings

Conversation

@maxtaran2010

Copy link
Copy Markdown

Fix several typos found by codespell in source files.

Changes:

  • control/robust.py: continous -> continuous (2 occurrences in TODO comments)
  • control/timeplot.py: tranpose -> transpose (comment)
  • control/nlsys.py: inpout -> input/output (comment), connecton -> connection (comment)
  • control/optimal.py: covert -> convert (comment)
  • control/phaseplot.py: Containter -> Container (docstring)
  • control/statesp.py: lamba -> lambda (comment)
  • control/flatsys/bspline.py: convertable -> convertible (error message string)

- robust.py: continous -> continuous (2 occurrences)
- timeplot.py: tranpose -> transpose
- nlsys.py: inpout -> input/output, connecton -> connection
- optimal.py: covert -> convert
- phaseplot.py: Containter -> Container
- statesp.py: lamba -> lambda
- flatsys/bspline.py: convertable -> convertible

Co-Authored-By: Claude <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 9, 2026 21:47

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.

@slivingston slivingston self-requested a review July 9, 2026 23:09
@slivingston

Copy link
Copy Markdown
Member

@maxtaran2010 Can you update tests to use the spelling-corrected strings?

@slivingston slivingston left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! Tasks before this can be merged:

  • Clarify authorship. You marked the commit with co-authored-by Claude, but these changes are easy to discover without Claude. Also, the PR description refers to codespell (I assume https://github.com/codespell-project/codespell). Can you confirm that Claude should be cited here instead of codespell? If you used codespell, please cite it in the commit message.

  • One of the assumed misprints is not one (cf. my in-line comment).

  • Please update tests.

Comment thread control/nlsys.py
iosys_repr(sys, format='info'), width=78,
initial_indent=" * ", subsequent_indent=" ")) + "\n"

# Build a list of input, output, and inpout signals

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is not a misprint. Please look at the context.

@slivingston

Copy link
Copy Markdown
Member

@maxtaran2010 Can you also find typos in other parts of the sourcetree? I did a quick search and noticed "continous" and "convertable" also appear in test files. Try doing this with grep instead of Claude ;)

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.

3 participants