Skip to content

Fix :param names in Twist3.UnitRevolute, Twist3.exp and Line2.contains - #277

Open
mmporong wants to merge 1 commit into
rai-opensource:masterfrom
mmporong:fix-param-names
Open

mmporong wants to merge 1 commit into
rai-opensource:masterfrom
mmporong:fix-param-names

Conversation

@mmporong

Copy link
Copy Markdown

Three docstrings use :param:/:type: names that do not match the signature:

  • Twist3.UnitRevolute(a, q, pitch=None) documents p.
  • Twist3.exp(theta=1, unit="rad") documents units.
  • Line2.contains(p, tol=20) documents p1.

This renames them to pitch, unit and p; the descriptions are unchanged. I found them with an automated comparison of the :param: entries in spatialmath/ against the function signatures.

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.

1 participant