Dense Worksheet Representation #34

New Issue

2013-09-09T20:38:10Z

schleumer commented

2013-09-09 20:38:10 +00:00

(Migrated from github.com)

You guys can make a option to choose range type, A-1 notation or [Sheet][Line][Column], it would be nice 👍

You guys can make a option to choose range type, `A-1` notation or `[Sheet][Line][Column]`, it would be nice :+1:

schleumer commented

2013-09-09 22:16:08 +00:00

(Migrated from github.com)

Well, i'm making a fastest-as-possible xlsx search, i got 2 files and each one had like 10 sheets, when a cell match with search the sheet and line which cell is present is returned, i made a wrapper of your lib which treat that for me, but i think it would be helpful as a implementation.

SheetJSDev commented

2014-06-30 00:15:54 +00:00

(Migrated from github.com)

@schleumer Are you walking in row-major (A1,B1,...,??1,A2,B2,...) or column-major (A1,A2,...,A##,B1,B2,...) order?

SheetJSDev commented

2017-04-08 07:00:34 +00:00

(Migrated from github.com)

@schleumer we didn't flip the default, but if you pass dense:true as an option to the various read functions (XLSX.read, XLSX.readFile, or any of the utilities that generate a worksheet or workbook) the generated worksheets are arrays. The writers and various utils have been updated to handle both types of worksheets. We were surprised by the performance difference but are hesitant to change the default until we are sure it won't break older browser versions.

@schleumer we didn't flip the default, but if you pass `dense:true` as an option to the various read functions (`XLSX.read`, `XLSX.readFile`, or any of the utilities that generate a worksheet or workbook) the generated worksheets are arrays. The writers and various utils have been updated to handle both types of worksheets. We were surprised by the performance difference but are hesitant to change the default until we are sure it won't break older browser versions.

👍 1

Sign in to join this conversation.