SFTtech/openage

Parse nyan API objects in converter instead of hardcoding them

Open

#1,313 创建于 2020年9月7日

在 GitHub 查看
 (8 评论) (0 反应) (0 负责人)Python (1,135 fork)batch import
area: assetsgood first issuelang: cythonlang: python

仓库指标

Star
 (12,130 star)
PR 合并指标
 (平均合并 9天 23小时) (30 天内合并 1 个 PR)

描述

Required skills: Python, Cython, nyan, maybe C++

Difficulty: Medium

Currently, the asset converter stores the openage nyan API as hardcoded nyan objects. It would be much better if the converter could read the API objects from the filesystem with a parser. There would be two ways to do this. Either we implement a new Python parser for nyan or we extend libnyan's already existing C++ parser with a Python interface (related issue in nyan repository).

Further reading:

贡献者指南

研究方向
首先研究 openage/convert/service/read/nyan api loader.py 中当前硬编码的 API 对象。然后查阅转换器架构文档 doc/code/converter/architecture overview.md，以理解转换器如何加载 nyan 对象。接下来，考虑两种方法：要么为 nyan 实现一个 Python 解析器，要么扩展 libnyan 的 C++ 解析器并为其添加 Python 接口，可参考相关的 nyan 议题 #64。所选方法应在实施前与维护者讨论。
技术栈
pythoncpp
领域
backendtoolingdeveloper experience
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-2 天
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
较久未更新
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
PythonC++
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
35