SFTtech/openage

Parse nyan API objects in converter instead of hardcoding them

Open

#1.313 aberto em 7 de set. de 2020

Ver no GitHub
 (8 comments) (0 reactions) (0 assignees)Python (1.135 forks)batch import
area: assetsgood first issuelang: cythonlang: python

Métricas do repositório

Stars
 (12.130 stars)
Métricas de merge de PR
 (Mesclagem média 9d 23h) (1 fundiu PR em 30d)

Description

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:

Guia do colaborador

Direção de pesquisa
Comece estudando os objetos de API atualmente hardcoded em openage/convert/service/read/nyan api loader.py. Em seguida, revise a documentação da arquitetura do conversor em doc/code/converter/architecture overview.md para entender como o conversor carrega objetos nyan. Depois, considere as duas abordagens: implementar um analisador (parser) Python para nyan ou estender o analisador C++ do libnyan com uma interface Python, consultando o issue relacionado do nyan #64. A abordagem escolhida deve ser discutida com os mantenedores antes da implementação.
Pilha de tecnologia
pythoncpp
Domain
backendtoolingdeveloper experience
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PythonC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
35