fastexcel icon indicating copy to clipboard operation
fastexcel copied to clipboard

`Worksheet.isCellInMergedRanges` make write slow.

Open jilen opened this issue 2 years ago • 2 comments

For example 10000 rows x 20 cols x 100 merged regions There could be 10_000_000 Worksheet.isCellInMergedRanges iterations.

jilen avatar Dec 24 '21 06:12 jilen

Possible solution:

  • Just treat cells in merged region as normal cell while calc width
  • Use some data structure like r-tree instead of set.

jilen avatar Dec 24 '21 06:12 jilen

Thanks for your feedback. Indeed, we could use a better data structure for mergedRanges or rethink column width computation, which is not perfect anyway.

ochedru avatar Jan 17 '22 12:01 ochedru