SFTtech/openage

Parse nyan API objects in converter instead of hardcoding them

Open

#1313 aperta il 7 set 2020

Vedi su GitHub
 (8 commenti) (0 reazioni) (0 assegnatari)Python (1135 fork)batch import
area: assetsgood first issuelang: cythonlang: python

Metriche repository

Star
 (12.130 star)
Metriche merge PR
 (Merge medio 9g 23h) (1 PR mergiata in 30 g)

Descrizione

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:

Guida contributor

Direzione di ricerca
Inizia studiando gli oggetti API attualmente hardcoded in openage/convert/service/read/nyan api loader.py. Poi rivedi la documentazione dell'architettura del convertitore in doc/code/converter/architecture overview.md per capire come il convertitore carica gli oggetti nyan. Successivamente, considera i due approcci: implementare un parser Python per nyan o estendere il parser C++ di libnyan con un'interfaccia Python, facendo riferimento al relativo issue nyan #64. L'approccio scelto dovrebbe essere discusso con i maintainer prima dell'implementazione.
Tech stack
pythoncpp
Dominio
backendtoolingdeveloper experience
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
35