Remove gRPC code generation
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- Mezza giornata
- Idoneità per principianti
- 35/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- python
- Ambito
- build-system
Direzione di ricerca
The gRPC generation is wired into [build-system].requires in pyproject.toml, the build_grpc command in setup.py, and the [build_grpc] section in setup.cfg; start by reading those three. Replace them with a python -m grpc_tools.protoc --python_out --pyi_out step, keeping grpcio-tools. Done when a wheel listing shows *_pb2.py and *_pb2.pyi but no *_pb2_grpc.py, and CI installs from source and sdist. This depends on #195 and #230, so check whether those have merged first.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
No .proto defines a service, and nothing in src/ or tests/ imports *_pb2_grpc. Yet the 1.0.0 wheel ships et_def_pb2_grpc.py, and every build runs setuptools-grpc.
Task
- Remove
setuptools-grpcfrom[build-system].requires. - Remove the
build_grpccommand insetup.pyand[build_grpc]insetup.cfg. - Generate code with
python -m grpc_tools.protoc --python_out --pyi_outonly. Keepgrpcio-tools, pinned as in #195, because it bundles protoc. - Keep the import rewrite from #230 (Goal 3), and run it on the files the new step generates.
Acceptance criteria
- The wheel contains
*_pb2.pyand*_pb2.pyiand no*_pb2_grpc.py. Evidence: wheel file listing. - Installing from source and from the sdist still works. Evidence: CI run.
Depends on
#195 (Goal 1; both edit [build-system].requires) and #230 (Goal 3).
- Lingua principale
- Python
- Stelle
- 201
- Fork
- 87
- Merge medio
- 6g 43m
- PR unite (30g)
- 2
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi 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 mlcommons/chakra
-
Release notes from PR titlesAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
Add code formatting checkAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
Add `chakra_linter` to validate Chakra execution traces against a rule setForse già presa @ajbalogh l’ha presa oggi. Apertafeature
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 22/100
Tutte le issue di mlcommons/chakra
Issue simili
-
needs-area-label
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
I maintainer di solito rispondono entro 2 giorni
-
enhancement good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
Official-Space-AI/OpenPhysGraspGen#114 ·
I maintainer di solito rispondono entro 2 giorni
-
changelog investigate
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
ramnes/notion-sdk-py#441 ·
-
qa_bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
NVIDIA-NeMo/Gym#5644 ·
I maintainer di solito rispondono entro 2 giorni
-
BUG: catalog database credentials printed to logs at startupForse già presa @cjboyle l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
I maintainer di solito rispondono entro 1 giorno