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

Parsing a file with a large number of rows #31

Open
apitts opened this issue Oct 26, 2014 · 6 comments
Open

Parsing a file with a large number of rows #31

apitts opened this issue Oct 26, 2014 · 6 comments

Comments

@apitts
Copy link

apitts commented Oct 26, 2014

Parsing 5,000 rows takes a few seconds but 30,000 rows just seemed to hang / not work. I stopped it after a few minutes though. I would have thought time would scale linearly with the number of rows but that appears not to be the case?

@AlfonsoJS
Copy link

I have the same problem!
If I try to parse a file that has more than 5000 rows, I get this error:

.../node_modules/excel/node_modules/node-promise/promise.js:147
throw new Error("This deferred has already been resolved");

More than 11 months nothing changed, so is this module still alive?
Would be great if you can solve the the problem because its a great module!
I really appreciate any help you can provide.

@trevordixon
Copy link
Owner

Is it alive? It appears to be useful to some still, but nothing is being done to improve it.

You might have better luck with https://github.com/mgcrea/node-xlsx.

@beflori
Copy link

beflori commented Jan 23, 2016

Same issue. Seems that it doesn't scale linearly. Will try to get it work with another lib.

@trevordixon
Copy link
Owner

@beflori I know it's been a long time, but out of curiosity, do you (and did you at the time you were looking into this) know about https://github.com/SheetJS/js-xlsx? If so, why use excel.js instead of that? I haven't worked on this in a long time, and I wonder if I should just point people to that other option, or if there's something compelling about excel.js that I should make more clear or enhance.

@beflori
Copy link

beflori commented Aug 28, 2017

It worked for us with node-xlsx and I wasn't aware of a better lib back then. Parsing Excel files seems to be error prone.

@shrimpwagon
Copy link
Contributor

#52

Merged and fixed

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

5 participants