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

[Docs] Tweak example excel file to prevent csvkit crash during tutorial #1221

Closed
wants to merge 1 commit into from

Conversation

johnhkchen
Copy link

@johnhkchen johnhkchen commented Nov 16, 2023

tl;dr: I re-opened the tutorial file in excel and now in2csv likes it again.

Hello,

I was working through the tutorial problems and got stuck on step 1.3.

The file grabbed by curl -L -O https://raw.githubusercontent.com/wireservice/csvkit/master/examples/realdata/ne_1033_data.xlsx, for whatever reason, generates an error when I ran in2csv ne_1033_data.xlsx (as per the instructions).

ValueError: Worksheet is unsized, use calculate_dimension(forWorksheet is unsized, use calculate_dimensione=True)

I think the error is generated by openpyxl, but don't have the expertise to delve deeper. Luckily, opening and saving the excel file using excel seemed to have cleared up the issues, so I'm creating this PR to hopefully clear up a discrepancy between the tutorial instructions and actual results.

in2csv 1.3.0
Ubuntu + Python 3.11/12

Steps to reproduce:

jpmckinney added a commit to wireservice/agate-excel that referenced this pull request Nov 20, 2023
@jpmckinney
Copy link
Member

Hi, this is actually a bug in agate-excel. I've released agate-excel 0.4.1. You can upgrade with:

pip install -U agate-excel==0.4.1

@jpmckinney jpmckinney closed this Nov 20, 2023
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

Successfully merging this pull request may close these issues.

2 participants