chore(scripts): mark the shebang-carrying Python scripts executable
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 1/5
- Thời gian dự kiến
- Dưới một giờ
- Mức phù hợp với người mới
- 84/100
Hướng nghiên cứu
Xem xét tám tệp được liệt kê có chứa shebang trong scripts/ và so sánh mode của chúng với scripts/ci/.py và scripts/bench_mlxlm.py. Áp dụng bit thực thi trong index, sau đó chạy git ls-files -s scripts/**/.py để xác minh mọi script có chứa shebang đều là 100755. Kiểm tra ba tệp không có shebang và ghi lại liệu chúng vẫn không thể thực thi hay được chuẩn hóa.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Summary
Eight Python scripts under scripts/ carry a #!/usr/bin/env python3 shebang but are not executable in the index, so the direct invocation some of them document for themselves fails with "permission denied".
Background
The scripts/ci/*.py files (and scripts/bench_mlxlm.py) are already mode 100755, so the executable convention is established. The affected files sit at 100644 despite their shebang. scripts/bench_embeddings.py:9 documents its own invocation as scripts/bench_embeddings.py --bin target/release/mlxcel-server ..., which cannot run today.
Proposed Solution
Set the executable bit in the index with git update-index --chmod=+x (or chmod +x then commit) on the shebang-carrying files. Verify with git ls-files -s scripts/**/*.py.
Implementation Notes
- Non-executable despite a shebang:
scripts/bench_embeddings.py,scripts/benchmark_gemma3n_python.py,scripts/benchmark_llama3_python.py,scripts/benchmark_python.py,scripts/bridge_overhead_microbench_py.py,scripts/gather_qmm_shape_microbench.py,scripts/layer_shaped_microbench_py.py,scripts/requantize_mlp.py(confirmed100644for the sampled files). - Separately,
scripts/insert_apache_header.py,scripts/tools/internlm_rope_oracle.py, andscripts/tools/quantize_mtp_drafter.pyhave neither a shebang nor+x. Decide whether to normalize them or leave them aspython3 <file>only.
Acceptance Criteria
- Every
scripts/**/*.pywith a shebang is executable in the index - The three shebang-less files are either normalized or explicitly left alone with a note in the PR
Original Suggestion
Title: chore(scripts): make the shebang'd Python scripts executable
Ten Python scripts under scripts/ carry a #!/usr/bin/env python3 shebang but are not executable, so the invocation some of them document for themselves fails with "permission denied". Every scripts/ci/*.py (and scripts/bench_mlxlm.py) is +x, so the convention is established.
Evidence
Non-executable despite a shebang: scripts/bench_embeddings.py, scripts/benchmark_gemma3n_python.py, scripts/benchmark_llama3_python.py, scripts/benchmark_python.py, scripts/bridge_overhead_microbench_py.py, scripts/gather_qmm_shape_microbench.py, scripts/layer_shaped_microbench_py.py, scripts/requantize_mlp.py.
scripts/bench_embeddings.py:9 documents its own invocation as scripts/bench_embeddings.py --bin target/release/mlxcel-server ..., which fails today.
Separately, scripts/insert_apache_header.py, scripts/tools/internlm_rope_oracle.py, and scripts/tools/quantize_mtp_drafter.py have neither a shebang nor +x — decide whether to bring them in line or leave them as python3 <file>-only.
Suggested fix
git update-index --chmod=+x on the shebang'd files (or chmod +x + commit). Purely mechanical; verify with git ls-files -s scripts/*.py.
Acceptance criteria
- Every
scripts/**/*.pywith a shebang is executable in the index - The three shebang-less files are either normalized or explicitly left alone with a note in the PR
- Ngôn ngữ chính
- Rust
- Star
- 470
- Fork
- 54
- Merge trung bình
- 5 giờ 39 phút
- Pull request đã merge (30 ngày)
- 292
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 lablup/mlxcel
-
area:core priority:low status:ready type:chore
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 90/100
-
priority:low status:ready type:docs
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 95/100
-
docs(webpage): add a webpage/site README covering the pnpm/uv/zensical build and deploy contract Đang mởpriority:low status:ready type:docs
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 86/100
-
docs: CONTRIBUTING says clippy is not gated at PR time, but ci.yml runs -D warnings clippy on PRs Đang mởpriority:medium status:ready type:docs
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
-
priority:low status:ready type:docs
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 92/100
Tất cả issue của lablup/mlxcel
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
state:needs triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
zed-industries/zed#64680 · 2 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
RustPython/RustPython#8802 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
TheLarkInn/aipm#2390 ·