Update URLs and field names to match the new vega weather datasets#2310
Update URLs and field names to match the new vega weather datasets#2310joelostblom wants to merge 1 commit intovega:mainfrom
Conversation
|
All test failures are related to me using the new name for the seattle data set from altair-viz/vega_datasets#40, and should be fixed once that is merged. |
|
Thanks for this. What needs to happen to fix this issue:
I've not done that yet, because it's a ~half day of work to get right, and I've not had an open half day in half a year. |
|
I should say: thanks for the contribution. I really appreciate it. But we can't merge changes like this until 1 & 2 are done. |
|
Thank you for responding so quickly! I understand that you are super busy and I didn't want to stress, more just checking in if I could do something helpful for moving this forward. But it sounds like most things you mentioned are release related, which sounds a bit tricky to help with. |
|
fixed by #3859 |
This fixes the broken weather sections in the docs mentioned in #2213. I upated based on my understanding of altair-viz/vega_datasets#40.