skia-python / skia-python/skia-python

Refactor code organization

Aperta
#130 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

enhancement
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

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. 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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.