tools.plot: drop the set_window_title copy once Spatial Math has it (SMTB #263)
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- Dưới một giờ
- Mức phù hợp với người mới
- 25/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- python
- Lĩnh vực
- data-visualization
Hướng nghiên cứu
The fallback set_window_title copy and its try/except import are in roboticstoolbox/tools/plot.py; the call sites are PyPlot, PyPlot2 and Vehicle. Hold the work until Spatial Math PR #263 is merged and released, then raise the minimum version in pyproject.toml. Done means the fallback is deleted, the call sites import from spatialmath.base, and the existing plot tests still pass.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Spatial Math PR rai-opensource/spatialmath-python#263 adds spatialmath.base.set_window_title(fig, title).
RTB has its own copy in roboticstoolbox/tools/plot.py, which already uses SMTB's when it exists (try: from spatialmath.base import set_window_title). Once the minimum Spatial Math version in pyproject.toml has it:
- require that Spatial Math version
- delete the fallback copy and the
try/except - import
set_window_titlefromspatialmath.baseat the call sites (PyPlot,PyPlot2,Vehicle)
Review by 2026-10-28: check whether #263 is merged and released.
Reminder to trigger the same change on MVTB and bdsim (see the linked issues there).
- Ngôn ngữ chính
- C++
- Star
- 3.5k
- Fork
- 639
- Merge trung bình
- 1 ngày 22 giờ
- Pull request đã merge (30 ngày)
- 73
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của petercorke/robotics-toolbox-python
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
petercorke/robotics-toolbox-python#709 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
mobile: the occupancy grid should draw the planner background, and planners should agree on unitsCó thể đã có người làm @kVinsom đã nhận 1 ngày trước. Đang mởtech-debt
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 30/100
petercorke/robotics-toolbox-python#785 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
tech-debt
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 22/100
petercorke/robotics-toolbox-python#773 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
CI: a PR influx is serialised by 'require up to date' plus a 12 minute CI; consider a merge queueĐang mởtech-debt
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
petercorke/robotics-toolbox-python#758 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
needs investigation tech-debt
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 18/100
petercorke/robotics-toolbox-python#755 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của petercorke/robotics-toolbox-python
Issue tương tự
-
binding:web core docs p3
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
RunanywhereAI/runanywhere-sdks#1123 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
NVIDIA-RTX/NRI#277 ·
Maintainer thường phản hồi trong vòng 1 ngày