Speed up the Refresh Event (TB) action

PopGoesTheWza / swgoh-tb-sheets

SWGoH Territory Battles Spreadsheets

MIT License

7 stars 2 forks source link

Speed up the Refresh Event (TB) action #21

Closed PopGoesTheWza closed 6 years ago

PopGoesTheWza commented 6 years ago

Current code updates the TB sheet in a very inefficient manner, with multiple small edits.

Implementation design:

regroup edit operations within arrays of objects, describing the affected range, edit (setValue, setFontWeight, etc.) and data
At the end of the data processing, parse the array and issue 'in bulk" the spreadsheet edits.

PopGoesTheWza commented 6 years ago

First round of optimisation done and released. Additional performance improvement still possible.

PopGoesTheWza commented 6 years ago

I close this issue for last improvement were "good enough" I'll likely cram in more optimisation when I return to this part of code (intent is to open the Event system to evaluate more than LS/DS TBs)