Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

does not include trailing comma when last column is empty #3

Closed
katylava opened this issue Dec 20, 2010 · 6 comments
Closed

does not include trailing comma when last column is empty #3

katylava opened this issue Dec 20, 2010 · 6 comments

Comments

@katylava
Copy link

given the headers A,B,C and data stuff,more stuff, [empty] the csv output doesn't not include a comma after "more stuff".

@dilshod
Copy link
Owner

dilshod commented Dec 21, 2010

it works for me, please send me your xlsx file if it's possible

@katylava
Copy link
Author

@dilshod
Copy link
Owner

dilshod commented Dec 21, 2010

i have fixed it, give it a try

@katylava
Copy link
Author

Thanks for the quick fix. That works, with that file. Funnily, the actual file I'm trying to process has some empty fields in the first column as well, and sometimes it's adding an extra comma to the beginning instead of the end. I will create another sample xlsx (my real one has PII in it) and post again later.

@katylava
Copy link
Author

Ok, here is a new xlsx.

There are 5 sheets and you can see the output from xlsx2csv.py for each of them in that same gist: https://gist.github.com/750235

I was trying to find the simplest example to recreate the bug, but as you can see from the output, it's... inconsistent.

I can work around it by moving the columns around in Excel. I'm just glad to have something to convert to csv since Excel really sucks at it.

@dilshod
Copy link
Owner

dilshod commented Dec 21, 2010

try it again, it should work now

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants