dilshod / xlsx2csv

Convert xslx to csv, it is fast, and works for huge xlsx files
MIT License
1.66k stars 302 forks source link

Latest commit fails to convert this spreadsheet #166

Open tonyle72 opened 5 years ago

tonyle72 commented 5 years ago

The latest commit 89ffd7d fails to convert this file:

$ ./xlsx2csv.py -a regif_test.xlsx regif
$ ls regif
$ ./xlsx2csv.py -n params regif_test.xlsx params.csv
Sheet 'params' not found

Release 0.7.4 works as expected.

(Note that this is a macro file and as such the proper extension is .xlsm; however, the uploader does not support this extension so I had to rename it to .xlsx.)

dilshod commented 5 years ago

Are you sure sheet named 'params' really exists? It works for me

tonyle72 commented 5 years ago

Hi Dilshod,

Yes. I’ve used this spreadsheet with previous versions (including the most recent verified 0.7.4) and it works fine. There’s something particular about this spreadsheet; other multi-sheet files I’ve tried work fine.

Tony


Tony Le | Director of Verification MAUI Imaging Inc. | 256 Gibraltar Drive, Suite 110, Sunnyvale, CA 94089 ' (408) 910-5915 | 7 (408) 583-4104 | • tony.le@mauiimaging.commailto:tony.le@mauiimaging.com

This electronic message contains information from MAUI Imaging, Inc. The contents of this message may contain privileged, confidential or proprietary information that is intended for the intended recipient only. If you are not the intended recipient, please notify MAUI Imaging, Inc. immediately – by replying to this message or by sending an email to alicia.woo@mauiimaging.commailto:alicia.woo@mauiimaging.com – and destroy all copies of this message and any attachments without reading or disclosing their contents. Any disclosure, use, copying or distribution of this message or its attachments is strictly prohibited.

From: Dilshod Temirkhodjaev notifications@github.com Sent: Friday, February 1, 2019 7:53 PM To: dilshod/xlsx2csv xlsx2csv@noreply.github.com Cc: Tony Le Tony.Le@MAUIImaging.com; Author author@noreply.github.com Subject: Re: [dilshod/xlsx2csv] Latest commit fails to convert this spreadsheet (#166)

Are you sure sheet named 'params' really exists? It works for me

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/dilshod/xlsx2csv/issues/166#issuecomment-459932514, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANlhMWPo-TAk2nN3cV5zbszifsa6LWHZks5vJQuSgaJpZM4ae_F6.