`grpc_servicer` should use `*args` to pass positional parameters
还没有人认领这个 Issue。
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 48/100
- Issue 类型
- 重构
- 描述清晰度
- 描述清楚
- 活跃度
- 停滞
- 技术栈
- grpc, python
调研方向
从所引用行的 ni_measurementlink_service/_internal/grpc_servicer.py 开始，阅读 MeasurementService.configuration_parameter 如何描述位置参数，以及 grpc_servicer 当前如何按名称映射这些参数。将该位置参数路径替换为 *args，同时保留 decorator 的顺序和现有的 service 行为。
由索引模型根据 Issue 内容生成。
描述
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.
- 主要语言
- Python
- 星标
- 21
- 派生
- 20
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
ni/measurement-plugin-python 的其他 Issue
-
Clean up the unused `build_in_import_modules` dictionary in ni-measurement-plugin-client-generator未关闭tech debt
难度 1/5 1 小时以内 新手友好度 68/100
ni/measurement-plugin-python#979 ·
-
tech debt
难度 2/5 1-3 小时 新手友好度 65/100
ni/measurement-plugin-python#926 ·
-
AUTO Session Initialization Fails in Python Measurement Plugin with VISA Devices可能重新可做 @bkeryan 于 336 天前认领，目前没有进行中的 PR。 未关闭bug
ni/measurement-plugin-python#1263 · 1 条评论 · 已指派 1 人 ·
-
enhancement
难度 4/5 3-5 天 新手友好度 45/100
ni/measurement-plugin-python#1227 ·
-
tech debt
难度 3/5 1-2 天 新手友好度 45/100
ni/measurement-plugin-python#978 ·
查看 ni/measurement-plugin-python 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 72/100
huggingface/sentence-transformers#4186 ·
维护者通常 5 天内回复
-
documentation good first issue
难度 2/5 半天 新手友好度 72/100
sujithputta02/Esapay#13 ·
-
bot-triaged module: correctness (silent) module: mps module: NaNs and Infs triage review
难度 2/5 半天 新手友好度 62/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 62/100
SCIInstitute/SCIRun#2842 ·
维护者通常 3 天内回复