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

Spec version warnings #767

Closed
ankane opened this issue Oct 23, 2021 · 6 comments · Fixed by #790
Closed

Spec version warnings #767

ankane opened this issue Oct 23, 2021 · 6 comments · Fixed by #790
Assignees
Labels
released this feature has been released!

Comments

@ankane
Copy link

ankane commented Oct 23, 2021

Hi, when running the HTML from the Vega-Lite tutorial, vega-embed shows two warnings about the spec version.

The input spec uses Vega-Lite v5, but the current version of Vega-Lite is v5.1.1. vega-embed@6.18.2:21:8121
The compiled spec uses Vega v5, but current version is v5.20.2. vega-embed@6.18.2:21:8553

It shows the same warnings with the latest version of vega-embed (6.19.1). Since everything is v5, it seems like there probably shouldn't be any warnings.

@ankane
Copy link
Author

ankane commented Oct 23, 2021

There are also source map warnings for each of the libraries:

DevTools failed to load source map: Could not load content for https://cdn.jsdelivr.net/npm/vega-embed.min.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE
DevTools failed to load source map: Could not load content for https://cdn.jsdelivr.net/npm/vega-lite.min.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE
DevTools failed to load source map: Could not load content for https://cdn.jsdelivr.net/npm/vega.min.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE

which can be fixed by using the full path:

<script src="https://cdn.jsdelivr.net/npm/vega@5.20.2/build/vega.min.js"></script>
<script src="https://cdn.jsdelivr.net/npm/vega-lite@5.1.1/build/vega-lite.min.js"></script>
<script src="https://cdn.jsdelivr.net/npm/vega-embed@6.18.2/build/vega-embed.min.js"></script>

@p-himik
Copy link

p-himik commented Nov 12, 2021

Just want to add that the warnings are there even if the versions are exactly the same:

vega-embed.js:5639 The input spec uses Vega v5.21.0, but the current version of Vega is v5.21.0.

@domoritz domoritz self-assigned this Nov 12, 2021
@domoritz
Copy link
Member

Hmm, we get a crash in semver "TypeError: Right-hand side of 'instanceof' is not an object at new Range$a (http://localhost:3000/build/vega-embed.js:2712:19) at Object.satisfies$3 [as satisfies] (http://localhost:3000/build/vega-embed.js:3294:17) at guessMode (http://localhost:3000/build/vega-embed.js:5667:21) at _embed (http://localhost:3000/build/vega-embed.js:5802:20) at embed (http://localhost:3000/build/vega-embed.js:5736:20) at async run (http://localhost:3000/test-vl.html:55:24)"

@domoritz
Copy link
Member

This seems to be a bug in the bundler and not Vega-Embed. See npm/node-semver#381

@github-actions
Copy link

🚀 Issue was released in v6.20.1 🚀

@github-actions github-actions bot added released this feature has been released! and removed prerelease labels Nov 15, 2021
@ankane
Copy link
Author

ankane commented Nov 15, 2021

Thanks @domoritz, that fixed it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released this feature has been released!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants