sexytopo
sexytopo copied to clipboard
richsmith
syncTableWithSurvey major performance bottleneck
syncTableWithSurvey is incredibly slow because it a) creates a TextView object for every row of the table and b) recreates them all every time we call the method. I am seeing this take up to 10 seconds to run per sync event due to these issues.
It should be possible to use a RecyclerView to alleviate some of these issues but more reading is required
Yes, the table activity is horribly inefficient. It's just been well behind all the graph stuff in terms of priority to be looked at. By design you can do almost everything from the graph view though, so in theory you don't need to use it much. It would certainly benefit from some attention.
Appreciate the work you two are doing on this project at the moment. I'm still hanging out for much more flexible data table workflow and functionality as summarised in #102 (near the end) and linked posts. Not pushing too hard, my Windows devices are still hanging in there, just want to keep it not too far from front of mind...
As discussed in the PR, we can also avoid syncing activities that are not currently active.
This has been improved in the latest version by only updating the open activity (that is, plan, EE, or table) on incoming data.