SFTtech/openage

Parse nyan API objects in converter instead of hardcoding them

Open

#1 313 ouverte le 7 sept. 2020

Voir sur GitHub
 (8 commentaires) (0 réactions) (0 assignés)Python (1 135 forks)batch import
area: assetsgood first issuelang: cythonlang: python

Métriques du dépôt

Stars
 (12 130 stars)
Métriques de merge PR
 (Merge moyen 9j 23h) (1 PR mergée en 30 j)

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:

Guide contributeur

Direction de recherche
Commencez par étudier les objets API actuellement codés en dur dans openage/convert/service/read/nyan api loader.py. Puis, consultez la documentation de l'architecture du convertisseur à doc/code/converter/architecture overview.md pour comprendre comment le convertisseur charge les objets nyan. Ensuite, envisagez les deux approches : soit implémenter un analyseur (parser) Python pour nyan, soit étendre l'analyseur C++ de libnyan avec une interface Python, en vous référant à l'issue nyan associée #64. L'approche choisie doit être discutée avec les mainteneurs avant la mise en œuvre.
Stack technique
pythoncpp
Domaine
backendtoolingdeveloper experience
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
35