`grpc_servicer` should use `*args` to pass positional parameters
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
- 48/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- grpc, python
- Lĩnh vực
- backend-api-design
Hướng nghiên cứu
Bắt đầu trong ni_measurementlink_service/_internal/grpc_servicer.py tại các dòng được tham chiếu, và đọc cách MeasurementService.configuration_parameter mô tả các tham số vị trí cũng như cách grpc_servicer hiện ánh xạ chúng theo tên. Thay thế đường dẫn xử lý tham số vị trí đó bằng *args, đồng thời giữ nguyên thứ tự của decorator và hành vi hiện có của service.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Tech Debt
The MeasurementService.configuration_parameter decorator describes positional parameters in order, but grpc_servicer uses inspect.signature to look up the parameter names so that it can use **kwargs to pass them:
https://github.com/ni/measurementlink-python/blob/main/ni_measurementlink_service/_internal/grpc_servicer.py#L98
https://github.com/ni/measurementlink-python/blob/main/ni_measurementlink_service/_internal/grpc_servicer.py#L245
Passing positional parameters with *args would be simpler and have slightly lower overhead.
- Ngôn ngữ chính
- Python
- Star
- 21
- Fork
- 20
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
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 ni/measurement-plugin-python
-
Clean up the unused `build_in_import_modules` dictionary in ni-measurement-plugin-client-generatorĐang mởtech debt
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 68/100
ni/measurement-plugin-python#979 ·
-
tech debt
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
ni/measurement-plugin-python#926 ·
-
AUTO Session Initialization Fails in Python Measurement Plugin with VISA DevicesCó thể làm lại được @bkeryan đã nhận 336 ngày trước và không có pull request nào đang mở. Đang mởbug
ni/measurement-plugin-python#1263 · 1 bình luận · 1 người được giao ·
-
enhancement
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
ni/measurement-plugin-python#1227 ·
-
tech debt
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
ni/measurement-plugin-python#978 ·
Tất cả issue của ni/measurement-plugin-python
Issue tương tự
-
documentation good first issue
Độ khó 2/5 Nửa ngày Mức phù hợp với người mới 72/100
sujithputta02/Esapay#13 ·
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Flaky: credential-proxy admission metric test asserts a wall-clock bound it does not establishĐang mởneeds-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
gke-labs/kube-agents#3277 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
jbacalso24/mimry#291 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
good first issue
Độ khó 2/5 Dưới một giờ Mức phù hợp với người mới 82/100
eli-alvarado/us-watchlist-alerts#14 ·
Maintainer thường phản hồi trong vòng 1 ngày