skia-python / skia-python/skia-python
Refactor code organization
Nessuno ha ancora preso questa issue.
- Lingua principale
- Jupyter Notebook
- Stelle
- 323
- Fork
- 52
- Merge medio
- 2h 29m
- PR unite (30g)
- 1
Descrizione
Is your feature request related to a problem? Please describe.
Re-organize binding codes according to Skia header structure. Also rename stub class name in a consistent manner; e.g., from:
py::class_<SkColorFilter, sk_sp<SkColorFilter>, SkFlattenable> colorfilter(m, "ColorFilter");
to:
py::class_<SkColorFilter, sk_sp<SkColorFilter>, SkFlattenable> ColorFilter(m, "ColorFilter");
This is to improve code maintenance over skia version upgrades.
Describe the solution you'd like
In Skia, header files are organized under different directories by categories, such as include/core/*.h or include/gpu/*.h. Presumably, the python package should be also split according to this structure.
Describe alternatives you've considered
n/a
Additional context
n/a
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia mappando il codice di binding esistente rispetto alle categorie degli header di Skia in include/core/.h e include/gpu/.h. Esamina la denominazione attuale delle classi stub, incluso l’esempio ColorFilter, e definisci la struttura dei package e l’ambito della denominazione prima di modificarla. Il lavoro è completato quando i binding seguono la struttura degli header e i nomi delle classi stub sono coerenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- cpp, python
- Ambito
- computer-graphics
- Tipo di issue
- Refactoring
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 25/100