search

a1b10 / cl-xlsx

📜 Read XLSX files with Common Lisp
23 stars 2 forks source link

Only every other sheet is read #2

Closed slyrus closed 3 years ago

slyrus commented 4 years ago

If I have an xlsx file with Sheet1, Sheet2, and Sheet3, only Sheet1 and Sheet3 are read. Similarly, if I have 6 sheets, only sheets 1,3, and 5 are read. Happy to provide a sample xlsx file but it seems easy enough to create one that fails.

gwangjinkim commented 4 years ago

Can I have the sample xlsx file?

There will be problems if the content of the sheets is not a simple table. It is generated only for simplest form of tables.

On Tue, Nov 5, 2019 at 10:03 PM Cyrus Harmon notifications@github.com wrote:

If I have an xlsx file with Sheet1, Sheet2, and Sheet3, only Sheet1 and Sheet3 are read. Similarly, if I have 6 sheets, only sheets 1,3, and 5 are read. Happy to provide a sample xlsx file but it seems easy enough to create one that fails.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/a1b10/cl-xlsx/issues/2?email_source=notifications&email_token=AHYOWF7AAFAF53TW3EI33V3QSHNR7A5CNFSM4JJJFCGKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HXAQBJA, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHYOWF5WT4CGY3SOOGDVGZTQSHNR7ANCNFSM4JJJFCGA .

slyrus commented 4 years ago

This one shows the problem:

six-sheets.xlsx

gwangjinkim commented 4 years ago

Oh you are right. I will try to fix it.

On Tue, Nov 5, 2019 at 10:38 PM Gwang-Jin Kim gwang.jin.kim.phd@gmail.com wrote:

Can I have the sample xlsx file?

There will be problems if the content of the sheets is not a simple table. It is generated only for simplest form of tables.

On Tue, Nov 5, 2019 at 10:03 PM Cyrus Harmon notifications@github.com wrote:

If I have an xlsx file with Sheet1, Sheet2, and Sheet3, only Sheet1 and Sheet3 are read. Similarly, if I have 6 sheets, only sheets 1,3, and 5 are read. Happy to provide a sample xlsx file but it seems easy enough to create one that fails.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/a1b10/cl-xlsx/issues/2?email_source=notifications&email_token=AHYOWF7AAFAF53TW3EI33V3QSHNR7A5CNFSM4JJJFCGKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HXAQBJA, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHYOWF5WT4CGY3SOOGDVGZTQSHNR7ANCNFSM4JJJFCGA .

gwangjinkim commented 4 years ago

Fixed. Now it should work.

gwangjinkim commented 3 years ago

fixed.