apertium / apertium/apertium-python

Optimize wrappers

Offen
#52 11 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement
Vorherrschende Sprache
Python
Sterne
35
Forks
27
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

While our wrappers avoid the extra process overhead, I think they can still be improved.

e.g. we only need to run https://github.com/apertium/lttoolbox/blob/master/python/lttoolbox.i#L23-L24 one time for any given `Analyzer` object

On another note, we should also try to do something smarter instead of re-opening files everywhere (but this might be harder to get working).

@unhammer, is doing the first thing I mentioned (i.e. only loading the dictionary once) meaningful in terms of improving performance? Sounds to me like it would be.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne mit den referenzierten Zeilen 23-24 in python/lttoolbox.i und untersuche die Wrapper-Einstiegspunkte in diesem Repository. Miss das wiederholte Laden von Wörterbüchern für einen Analyzer sowie die Kosten des erneuten Öffnens von Dateien; done sollte durch Performance-Nachweise und Tests belegt werden, die das resultierende Wrapper-Verhalten abdecken.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
performance
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
30/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.