-
Notifications
You must be signed in to change notification settings - Fork 799
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
Update from Vega-Lite 5.7.1 to 5.8.0 #3037
Conversation
https://altair-viz.github.io/gallery/wind_vector_map.html now gives:
When defining a Feature, including a properties member it works: extent = {
"type": "Feature",
"geometry": {
"type": "Polygon",
"coordinates": [
[
[xmax, ymax],
[xmax, ymin],
[xmin, ymin],
[xmin, ymax],
[xmax, ymax],
]
],
},
"properties": {},
} |
I've updated the tests and documentation reflecting the changes in VL5.8. The change introduced was for the Are you able to update |
Yes, update is in the works in vega/vl-convert#55. I'll ping you here when everything is published (should be within a few hours if I don't hit any snags). |
Thanks, that is great🙏👍 |
Ok, |
Thanks @jonmmease, all tests pass, merging here👍 |
This updates Altair to use Vega-Lite 5.8.0.
One thing I notice in the diff-changes:
old, 5.7.1
new, 5.8.0
where https://altair-viz.github.io/gallery/wind_vector_map.html is using a
GeoJsonFeature
forfit
.Not sure yet if this a regression.
Also this depends on this PR at the altair_viewer repo altair-viz/altair_viewer#57 which awaits confirmation to be merged.