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

Blank charts due to iris url not working in user guide #2210

Closed
johnyrufus opened this issue Jun 18, 2020 · 6 comments
Closed

Blank charts due to iris url not working in user guide #2210

johnyrufus opened this issue Jun 18, 2020 · 6 comments

Comments

@johnyrufus
Copy link
Contributor

In the user guide:
https://altair-viz.github.io/user_guide/compound_charts.html#

The iris charts are blank, because the iris url is broken, as 404 is returned when trying to access

https://vega.github.io/vega-datasets/data/iris.json

or when trying to execute the following code

import altair as alt
from vega_datasets import data
iris = data.iris.url
print(iris)
import urllib.request, json 
with urllib.request.urlopen(iris) as url:
    data = json.loads(url.read().decode())
    print(data)
@johnyrufus johnyrufus changed the title blank charts dur to iris url not working in user guide Blank charts due to iris url not working in user guide Jun 18, 2020
@jakevdp
Copy link
Collaborator

jakevdp commented Jun 18, 2020

Thanks - it looks like vega-datasets removed the iris dataset in the most recent release: vega/vega-datasets#187

@jakevdp jakevdp transferred this issue from altair-viz/altair-viz.github.io Jun 18, 2020
@jakevdp
Copy link
Collaborator

jakevdp commented Jun 18, 2020

There were a number of other changes as well. We'll have to go through the docs and remove references to obsolete data sources.

@johnyrufus
Copy link
Contributor Author

Sure, I can take a look into it if thats okay.
Directly using data.iris() works though.

@jakevdp
Copy link
Collaborator

jakevdp commented Jun 18, 2020

Directly using data.iris() works though.

That will work until the Python vega_datasets package is updated, at which point iris will be removed following the upstream package.

@johnyrufus
Copy link
Contributor Author

Makes sense. Thanks for the info.

@joelostblom
Copy link
Contributor

@johnyrufus I am going through Altair issues to find those that have been resolved and can be closed. It looks to me like this issue is a duplicate so I am closing it, but please feel free to reopen and add a comment if you don't think this is correct.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants