- Notifications
You must be signed in to change notification settings - Fork1.9k
Fix some xlsx stream read xlsx not in guaranteed order problem#578
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
guyonroche commentedJun 26, 2018
@KMethod my concern with this PR is the use of Temp, which won't work in a browser. The StreamBuf class (in utils) should provide an in-memory alternative |
KMethod commentedJun 28, 2018 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@guyonroche Streaming XLSX are only exposed in NodeJS, so does it matter？ |
guyonroche commentedSep 5, 2018
@KMethod Sorry this is dragging out - I don't have a huge amount of time these days. I'm still not sure what problem this fix is solving. Can you give more details? |
when stream read some xlsx files (like numbers convert to xlsx), get values like :
row.values [ ,
{ sharedString: 47 },
{ sharedString: 35 },
{ sharedString: 47 },
{ sharedString: 37 },
{ sharedString: 37 },
{ sharedString: 37 },
{ sharedString: 37 },
{ sharedString: 37 },
{ sharedString: 37 },
]
try to fix this problem