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

Notebook tooling: clear_outputs.py stops at the first dirty cell, wasm_check does not test the built wheel

Đang mở
  • Có gì không ổn?
#157 0 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

Chưa có ai nhận issue nà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
42/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
jupyter, python, wasm
Lĩnh vực
testing, tooling

Hướng nghiên cứu

Start with docs/notebooks/clear_outputs.py and its process_notebook function, then inspect docs/lite/wasm_check/check_pyodide_install.mjs alongside _mvtb_nb_bootstrap.py. Check that every dirty cell is cleared and that the wheel is written where the bootstrap searches; the issue says both fixes are already present in the RTB copy, so use its notebook hygiene PR as a reference. Done when both tools operate on all dirty cells and the wheel built for the check.

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

Mô tả

Likely smells in the notebook tooling, found while copying it to RTB. Worth checking out and fixing.

  1. docs/notebooks/clear_outputs.py, process_notebook: changed = any(clear_cell(cell) for cell in nb.get("cells", [])) is a generator, and any() stops at the first cell that returns True. Only the first dirty cell of each notebook is cleared, so a second run clears the next one, and so on. Use a list: any([clear_cell(c) for c in cells]).

  2. docs/lite/wasm_check/check_pyodide_install.mjs writes the wheel to /<wheel> in Pyodide's filesystem, but _mvtb_nb_bootstrap.py looks for it with Path("/pypi").glob(...). It finds nothing, so it falls back to installing machinevision-toolbox-python from PyPI and the check tests the last release, not the wheel just built. Also write the wheel to /pypi/<wheel> (FS.mkdirTree("/pypi")).

Both are fixed in the RTB copy, see the notebook hygiene PR in robotics-toolbox-python.

Ngôn ngữ chính
Python
Star
224
Fork
31
Merge trung bình
1 ngày 20 giờ
Pull request đã merge (30 ngày)
28

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/machinevision-toolbox-python

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

Issue tương tự

Thêm issue về Python

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.