-
Notifications
You must be signed in to change notification settings - Fork 794
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
feat: Upgrade Vega-Lite from 5.17.0 to 5.19.0 #3479
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Setting "request changes" to flag that vl-convert doesn't support Vega-Lite 5.19.0 yet.
See #3478 (comment)
Oh, good point! I somehow stopped half-way through the instructions, sorry for that :) I've now also bumped the version used in JupyterChart. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Heh, actually I was confused by my own release notes. Vega-Lite 5.19.0 is already released in vl-convert 1.5, so nothing is blocking there after all!
This PR looks good to go to me!
Oh, actually, I think we still need to update the vl-convert version in the importer here: altair/altair/utils/_importers.py Line 44 in 9afd374
|
I've now updated the vl-convert version in |
Is this issue now appearing #3447? |
In the first commit, I only changed the version number and ran the code generation. The other commits are manual changes.