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.

Sign up for GitHub

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

Jump to bottom

Running the contributed command:'map.preview' failed. #28

Closed
smcardle opened this issue Dec 13, 2018 · 4 comments · Fixed by #65
Closed

Running the contributed command:'map.preview' failed. #28

smcardle opened this issue Dec 13, 2018 · 4 comments · Fixed by #65

Comments

@smcardle
Copy link

Opening a GeoJSON file then selecting map preview causes the following error to display

Running the contributed command:'map.preview' failed.

The GeoJSON file is 103.5 MB and was exported from mapshaper.org

Steve
@jumpinjackie
Copy link
Owner

The GeoJSON file is 103.5 MB

o_O

I don't think this extension was designed to preview files of that size

@netotz
Copy link

netotz commented Nov 17, 2020

same problem, my file is 139 MB

@szewah
Copy link

szewah commented Feb 24, 2021

Same problem. My file is 1.32GB

@jumpinjackie
Copy link
Owner

Sorry, but VSCode's extension APIs will not cooperate with us when working with files over a certain limit (currently 50MB). Such files we cannot get important information about the open document to initiate a map preview.

Until VSCode improves their extension API support around large files, there is nothing I can do here: microsoft/vscode#31078

For the next 0.6.0 release, we will show the error message This file is too big to be previewed if attempting to map preview such a large file.

@jumpinjackie jumpinjackie closed this as not planned Won't fix, can't repro, duplicate, stale Sep 11, 2023
jumpinjackie added a commit that referenced this issue Sep 11, 2023
@jumpinjackie
#28: The VSCode extension API will not tell us details about the acti…
a4fb4c2 
…ve document if it is over 50MB in size. We cannot do anything about this, so just show an appropriate error message if we cannot acquire the active text editor.
@jumpinjackie jumpinjackie mentioned this issue Sep 11, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

More QoL improvements jumpinjackie/vscode-map-preview
4 participants
@jumpinjackie @smcardle @netotz @szewah