search

mechatroner / vscode_rainbow_csv

🌈Rainbow CSV - VS Code extension: Highlight CSV and TSV files in different rainbow colors to make them more readable
MIT License
432 stars 53 forks source link

Fails if file too big? #153

Closed AJamesPhillips closed 12 months ago

AJamesPhillips commented 1 year ago

Hi, I love this tool. Really great. Thank you. When I open a small file it works fine but with larger files it doesn't seem to load the extension command UI buttons:

Cursor_and_2019Q4_csv_—_housing-data

The rainbow colours for the file contents renders fine (as you can see above) so the file must have been read and parsed ok.

The files in question are from here: https://statistics.gov.scot/data/domestic-energy-performance-certificates and are about 50Mb each so not that large. If I make a subset of the file it works fine. I thought it might be some bad character which might be breaking the extension but then the rainbow colours work so I don't know.

mechatroner commented 1 year ago

Thanks, I appreciate the feedback! There is a VSCode limitation unfortunately that prevents extension access to files larger than 50 MB, see https://github.com/microsoft/vscode/issues/31078, Rainbow CSV just can't see such files. But highlighting does work apparently because it is implemented as a separate texmate grammar file.

AJamesPhillips commented 1 year ago

Thanks so much for the speedy reply @mechatroner ! Do you think it would be possible to show a message in the logs or something for the user to know the extension has not loaded due to the large file size?

mechatroner commented 12 months ago

I've subscribed to the VSCode issue I linked to know if there are any API updates to handle this situation better. Currently, big files are completely hidden from extensions - there is no reliable way to request big file size, filename, extension, filetype, etc. So there is no way to show the message in the UI, and I don't think logging something to the debug log would have a lot of value (and again it won't probably be precise enough)

AJamesPhillips commented 12 months ago

Ah got it. Thank you. Ok I'll close this. I hope they improve the API around this.