chore(scripts): mark the shebang-carrying Python scripts executable
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 1/5
- Tempo stimato
- Meno di un'ora
- Idoneità per principianti
- 84/100
Direzione di ricerca
Esamina gli otto file elencati che contengono uno shebang in scripts/ e confronta le loro modalità con scripts/ci/.py e scripts/bench_mlxlm.py. Applica il bit di esecuzione nell’indice, quindi esegui git ls-files -s scripts/**/.py per verificare che ogni script contenente uno shebang sia 100755. Controlla i tre file senza shebang e annota se rimangono non eseguibili o vengono normalizzati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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
- Lingua principale
- Rust
- Stelle
- 470
- Fork
- 54
- Merge medio
- 4h 45m
- PR unite (30g)
- 305
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di lablup/mlxcel
-
area:core priority:low status:ready type:chore
Difficoltà 2/5 1-3 ore Idoneità per principianti 90/100
-
priority:low status:ready type:docs
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 95/100
-
docs(webpage): add a webpage/site README covering the pnpm/uv/zensical build and deploy contract Apertapriority:low status:ready type:docs
Difficoltà 1/5 1-3 ore Idoneità per principianti 86/100
-
docs: CONTRIBUTING says clippy is not gated at PR time, but ci.yml runs -D warnings clippy on PRs Apertapriority:medium status:ready type:docs
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
-
priority:low status:ready type:docs
Difficoltà 2/5 1-3 ore Idoneità per principianti 92/100
Tutte le issue di lablup/mlxcel
Issue simili
-
Browser (wasm) relay client cannot connect to relays whose URL has a trailing-dot FQDN hostname Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
n0-computer/iroh#4550 ·
-
impl detach for native Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
paritytech/zombienet-sdk#591 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
farion1231/cc-switch#7638 · 1 commento ·
-
onnx-ir re-exports ModelProto and GraphProto but not NodeProto, AttributeProto and AttributeType Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100