plt.pause(0) waits for ever: PoseGraph.plot_scan(pause=0) and Vehicle animation may hang
Maintainer thường phản hồi trong vòng 1 ngày
Đá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ả
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,pauseis an argument and defaults to 1. Needs a guard, or aValueError, as in #875. -
Vehicleanimation (mobile/Vehicle.py,_refresh_animation_frame(fig, dt, pause)): ends withplt.pause(dt); check whetherdtcan 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
- 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
-
tech-debt
Độ khó 4/5 1-2 ngày Mức phù hợp với người mới 55/100
petercorke/robotics-toolbox-python#898 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
LatticePlanner: integer headings, and motion primitives with finer turn steps (design first)Có thể đã có người làm @kVinsom đã nhận 1 ngày trước. Đang mởenhancement help wanted
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
petercorke/robotics-toolbox-python#895 · 1 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 18/100
petercorke/robotics-toolbox-python#894 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Backends wanted: CoppeliaSim, Viser, Gazebo, MuJoCo and others (plan first)Có thể đã có người làm @alexermolovich đã nhận 2 ngày trước. Đang mởenhancement help wanted
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 38/100
petercorke/robotics-toolbox-python#886 · 1 bình luận ·
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ự
-
Remove source files from packaging for deb, zip, etcCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
OpenSpace/OpenSpace#4353 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 5 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
justcallmekoko/ESP32Marauder#1656 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
-
Decompiler: convertCharRef() in xml.y overflows its int4 accumulator on long character referencesCó thể đã có người làm @kaayzouee đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
NationalSecurityAgency/ghidra#10009 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
mutationpp/Mutationpp#337 ·