-
-
Notifications
You must be signed in to change notification settings - Fork 59
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
no preview for csv file #250
Comments
@PierreMarchand20 do you see any errors in Help -> Toggle Developer Tools when you run it? |
yeah, I need to look into this. That updateStats call just reports the number of records read from a data file and size in vscode status bar and ext. log. Thank you for sharing your error outputs. |
Let me know if you need something else. Your plugin, after Jupyter Notebooks, is the key reason to recommend VSCode... Coffee inbound. |
Exact same error as @PierreMarchand20 and @mrsoftowl over here ! |
Hi there, as far as I find, the error is caused by the dark theme. Change the theme to |
Changing theme has no effect on my installation |
so, after updating to vscode v1.53.2 I can finally reproduce this for a dark data preview theme. I was not able to do it with the initial v1.53 release. Also, interestingly enough an older data preview I had open for zipcodes while testing with earlier version of that vscode release worked, while creating new data preview after vscode update to latest did not. Oddly enough, changing data preview to If you want to try it, just change data preview theme setting in Preferences -> Settings -> Extensions -> Data Preview -> Theme or set it in I'll see if I can debug and patch this soon. It's a bit tricky and I suspect there is more to it than that error we get. Thanks for your patience. |
I am going to try this with new vscode v1.54 release today. I was not able to find the origin of this issue for the dark theme and that css is not much different from the stylesheet I am using for light data preview theme. I do suspect new built-in vscode table widget or something else related to webview loading has something to do with it since after fixing that stats update call I saw some Table trace logs that I did not recognize. Will try it in the latest vscode v. and keep you posted. |
Both light and dark scheme fail randomly for me, with light theme seeming to work a bit more reliably. (light loaded data 6 out of 10 tries, dark loaded it 4 out of 10 tries). FWIW, I get a similar but slightly different error as well. It might just be a linux thing though (vscode v1.54.1).
|
Same problem for me, VS Code 1.54.1. Does not load, however it accurately shows number of rows and size in the bottom bar |
Similar error for me with a .csv |
Same here. Light theme works with no error but dark theme failed with error:
|
Can confirm that the extension will not preview CSV with theme Dark, but does work with theme: Light, and the first clue that I notice is |
Hi, amazing extension! But sadly, I'm also getting preview issues with a large file (+26MB) in both light and dark themes. Refreshing the data in VS Code does not help much... I was able to get the data to load once, but I haven't been able to preview it since... at least 50 tries of refreshing and/or reopening the file. In dev tools, i get similar errors as above. In the lower status bar, shows
|
well, it actually works on gitpod.io. So, I suspect there is a conflicting extension that stops it from working, or some faulty vscode update in the past two releases. works with CSV files too. Try Data: Preview Remote data command and paste this data url: https://us-counties.datasette.io/counties/counties.csv |
PS: this is the problem and it's not coming from my extenion. something else steals data transfer to my webview:
|
@matrix-ranger have you tried it with the Just set it to that for now, till I find the time to look into those data transfers issues. I suspect they are related to new changes for byte array transfers: https://code.visualstudio.com/updates/v1_56#_improved-transfer-of-arraybuffers-to-and-from-webviews I was hoping vscode v1.56 release would fix it. I did not have time to investigate this further. There is def. something very odd at the core going on there. |
The light themes work well. And btw I am using vscode v1.56. |
some devs experiencing this issue reported that prior version of DataPreview v2.2.0 worked. Please try that version if you run into this issue on your vscode or OS. You can download it from here: https://marketplace.visualstudio.com/items?itemName=RandomFractalsInc.vscode-data-preview&ssr=false#version-history Not much changed since that version in 2.3.0. I am still not sure what's causing this. Could be new WASM loading or other general vscode files reading issues on Darwin OS? #250 (comment) See them here for more info: https://github.com/microsoft/vscode/issues?q=is%3Aissue+WASM+Darwin Please report your OS and version, if you run into this. |
well, I can confirm that the last published Data Preview v2.3.0 works in vscode v1.56.0 on win7: It also works in Code Spaces. See: RandomFractals/vscode-leaflet#9 (comment) and in the latest vscode v1.56.2: |
It works only in the light theme for me with no errors in the dark theme but failing to load. Version: 1.56.2 (Universal) |
that's good info. I tested it with @alefragnani last week and we narrowed it down to Darwin OS so far. Still doesn't explain why switching data preview theme stylesheet would break or fix that data transfer and display. There are quite a few issues related to extension host reload and wasm in vscode since v1.54.0 release. I might bite the bullet and update that grid lib and dependencies and bundle this extension to trim it down a bit. Other than that I am still puzzled by some of those errors. Thanks for your findings @mhkeller. Much appreciated! |
I just learned you can get some more debug output from the
versus light mode
|
yeah, that provides some debug info. for some reason data is read from file in those blank grid cases, but never makes it to the data preview webview. |
Hey, this is an awesome extension =) Unfortunately, I'm facing this issue too. Changing to white theme seemed to work. Maybe the issue should be reopened for visibility? I almost opened one myself because I couldn't find anything related on the open ones. |
@eduardocanellas I agree. This started with a trickle of random reports and turned into a major issue on Darwin OS now. I am going to reopen it and wait for vscode v1.57.0 release coming soon that will be using different approach for bytes data transfer. Please use the light stylesheet for now and ease up on the negative 1 star reviews until we find a solution. Thanks! |
so, vscode v1.57.0 is out. Can you try loading Data Preview with the dark theme and that version of vscode to see if this tricky webview data loading issue went away? |
@RandomFractals Unfortunately, the problem persists here in 1.57.0. With the dark theme enabled, sometimes it works, sometimes it doesn't. FWIW, the light theme has been working great since I enabled it. |
@eduardocanellas thanks for a quick check. There is this bit in release notes, but that would only affect arrow data that I transfer via bytes array, and not other data file types. https://code.visualstudio.com/updates/v1_57#_improved-webview-array-buffer-transfers I still suspect it's related to wasm and webviews loading on Macs. There are just too many issues open in vscode repo for me to pin it down, and unfortunately I don't have a Mac to test it. Maybe defaulting to light theme is all we can do for a quick fix and new users trying it on a Mac. |
My OS is Linux, by the way. POP OS, to be more precise. |
I get the same behavior in dark mode. light mode works fine
|
so, yeah, I think I'll change default to light theme next week and maybe see if updating engines to v1.57.0 will fix things for the dark theme. Thank you guys for trying it again in the latest vscode release v. |
also, webviews are less than stable once again in the latest release and a lot of devs are hitting webviews not loading issue: Please be aware of that. |
Just chiming in saying this is one of my most favorite extensions but since the regressions a few months ago has become hard to use. I have found light theme to be much more reliable than dark theme. I am on 2.3.0 for Data Preview and 1.58.2 for VS Code |
@uditrana yes, please use the light data preview theme. Thanks for chiming in. |
Chalk another one up for this; I want to look at a comma delimited CSV, but i'm getting the 'Open Data file to Preview' toast. I'm using the light theme, and there are no errors in the dev console. |
@hmccurryWM try our Tabular Data Viewer from last year. It can load larger CSV files and uses more recent viewer v.: https://marketplace.visualstudio.com/items?itemName=RandomFractalsInc.tabular-data-viewer |
Sorry for the naive question, but I cannot preview a simple csv file:
Am I missing something ? here is the log
The text was updated successfully, but these errors were encountered: