Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

plt.pause(0) waits for ever: PoseGraph.plot_scan(pause=0) and Vehicle animation may hang

Đang mở Phù hợp với người mới
#876 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@huxiaoyi-ovo đang làm issue này rồi.

Từ ngày 23/10/2026.

  • #896 của @huxiaoyi-ovo — đang mở

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
72/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
matplotlib, python

Hướng nghiên cứu

Đọc mobile/PoseGraph.py tại PoseGraph.plot_scan và mobile/Vehicle.py tại _refresh_animation_frame; kiểm tra các vị trí gọi ở dòng 853 và 1841 để xác định liệu dt có thể bằng không hay không. So sánh cách xử lý trong PlannerBase.animation_frame và cách tiếp cận kiểm thử timeout trong tests/test_step_dt.py của #875. Bổ sung kiểm thử cho các giá trị tạm dừng bằng không và xác nhận các lời gọi bị ảnh hưởng sẽ trả về hoặc phát sinh ngoại lệ thay vì bị treo.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

bug

plt.pause(0) does not return: Matplotlib treats a timeout of zero as "wait until the event loop is stopped", so it blocks for ever, with no message. PyPlot.step(0) and PyPlot2.step(0) did this and now raise ValueError (#875). Other places pass a caller's value straight to plt.pause:

  • PoseGraph.plot_scan(n, pause=0) (mobile/PoseGraph.py): plt.pause(pause) with no guard, pause is an argument and defaults to 1. Needs a guard, or a ValueError, as in #875.
  • Vehicle animation (mobile/Vehicle.py, _refresh_animation_frame(fig, dt, pause)): ends with plt.pause(dt); check whether dt can be zero from the callers (lines 853 and 1841) and guard it if so.

Not affected: PlannerBase.animation_frame already tests pause > 0, and the others pass a constant.

I believe the first two hang the same way but have not run them. A test should use a timeout so that a regression fails instead of hanging (see tests/test_step_dt.py in #875).

Ngôn ngữ chính
C++
Star
3.6k
Fork
645
Merge trung bình
1 ngày 8 giờ
Pull request đã merge (30 ngày)
144

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của petercorke/robotics-toolbox-python

Tất cả issue của petercorke/robotics-toolbox-python

Issue tương tự

Thêm issue về C++

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.