Skip to content

Fixes to base.plot_box() - #179

Open
petercorke wants to merge 1 commit into
rai-opensource:masterfrom
petercorke:fix/base-plot-box
Open

Fixes to base.plot_box()#179
petercorke wants to merge 1 commit into
rai-opensource:masterfrom
petercorke:fix/base-plot-box

Conversation

@petercorke

Copy link
Copy Markdown
Collaborator

Improve handling of the "format string" arg.
Provide a US-spelling version of the 'centre' option, either 'center' or 'centre' will work.
Prevent multiple Matplotlib arguments of the same name being passed to plt.Rectangle, as happens from plot_labelbox. Improve the docstring.

Minor fix in docstring for plot_arrow, tex string should be raw. This upsets Sphinx when any other package imports spatialmath.

Provide a US-spelling version of the 'centre' option, either 'center' or 'centre' will work.
Prevent multiple Matplotlib arguments of the same name being passed to `plt.Rectangle`, as happens from `plot_labelbox`.
Improve function docstring.
Minor fix in docstring for `plot_arrow`, tex string should be raw.  This upsets Sphinx when any other package imports spatialmath.
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 58.33333% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
spatialmath/base/graphics.py 58.33% 5 Missing ⚠️

📢 Thoughts on this report? Let us know!

petercorke added a commit to petercorke/spatialmath-python that referenced this pull request Jul 27, 2026
Comprehensive handover tracking:
- Original problem statement (PR rai-opensource#179 stuck)
- Full plan with 4 phases
- Action to date (3 PRs pushed/ready)
- Action going forward (next 3 branches + guidelines)
- Status table and next-session checklist

This document serves as the source of truth for PR work state
and continuation instructions for future sessions.
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