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

example_18_06_XMLYahooWeather out of date #200

Open
ybakos opened this issue May 9, 2016 · 1 comment
Open

example_18_06_XMLYahooWeather out of date #200

ybakos opened this issue May 9, 2016 · 1 comment

Comments

@ybakos
Copy link

ybakos commented May 9, 2016

The Yahoo weather API has recently changed, and this example needs a little massaging.
Ref: processing/processing#4422

Will send a minimal PR that shouldn't require too much change to the text. Biggest difference is the need to introduce the concept of a "woeid".

ybakos pushed a commit to ybakos/LearningProcessing that referenced this issue May 9, 2016
@shiffman
Copy link
Owner

shiffman commented May 9, 2016

Thanks so much for this fix! I'm going to re-open this issue as I'll need to change the book text itself as well as some point. (In a different repo that I am not allowed to make public as much as I have tried.)

@shiffman shiffman reopened this May 9, 2016
@shiffman shiffman reopened this Jul 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants