PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
cache power reports
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Lua
- Sterne
- 5.4k
- Forks
- 2.4k
- Ø Merge
- 1 T. 12 Std.
- Gemergte PRs (30 T.)
- 26
Beschreibung
Check version
- I'm running the latest version of Path of Building and I've verified this by checking the changelog
Check for duplicates
- I've checked for duplicate issues by using the search function of the issue tracker
Is your feature request related to a problem?
Power reports are extremely slow to generate, and they are often switched between even without modifying anything.
Describe the solution you'd like
A very simple cache would wok wonders for many use cases. You don't even need to be smart about caching sub-graph, just invalidate on modification. You could also keep a map of the build code to results for when edits are done and reverted often. I do that a lot, but I'm not sure how others use it. These are both pretty low impact changes I would think.
Describe alternatives you've considered
Rewrite in c++ probably isn't a great solution.
Additional context
Also, some progress indicator in a window footer for how much computation still needs to be done would be nice.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Das Issue nennt keine Dateien, Tests oder einen Einstiegspunkt. Beginne damit, den Ablauf zur Generierung von power-report zu lokalisieren und zu bestimmen, wie Änderungen am Build erkannt werden; bewerte anschließend Caching und den angeforderten Fortschrittsindikator getrennt. Als abgeschlossen gilt die Aufgabe, wenn wiederholte, unveränderte Reports unnötige Berechnungen vermeiden und die verbleibende Arbeit durch geeignete Validierung abgedeckt ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- lua
- Bereich
- performance
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100