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

Add weather_wunderground.pl to support Weather Underground data #617

Merged
merged 10 commits into from
Nov 21, 2016

Conversation

pmatis
Copy link
Contributor

@pmatis pmatis commented Sep 18, 2016

I thought I created a pull request long ago, but I guess not.

@hplato
Copy link
Collaborator

hplato commented Sep 18, 2016

Hi Steve,

Hey only one thought on this. The calc_eto stuff I did also pulls from weather underground, I use the following config_parms:

eto_location : not just a stationID, since I can find the nearest one if given lats and longs
wu_key : api key

Might it make sense to have common config parameters?

On Sep 18, 2016, at 11:50 AM, Steve Switzer [email protected] wrote:

I thought I created a pull request long ago, but I guess not.

You can view, comment on, or merge this pull request online at:

#617 #617
Commit Summary

First creation of common/weather_wunderground.pl
fix weather station change
Fix trigger creation, add debug option.
Adding weather...
Merge branch 'wunderground' of github.com:pmatis/misterhouse into wunderground
Merge branch 'master' of github.com:hollie/misterhouse into wunderground
Merge remote-tracking branch 'remotes/hollie/master' into wunderground
Updating to version that I'm using, cleaned up code, add api_key detection.
Merge remote-tracking branch 'hollie/master' into wunderground
.
File Changes

A code/common/weather_wunderground.pl https://github.com/hollie/misterhouse/pull/617/files#diff-0 (155)
Patch Links:

https://github.com/hollie/misterhouse/pull/617.patch https://github.com/hollie/misterhouse/pull/617.patch
https://github.com/hollie/misterhouse/pull/617.diff https://github.com/hollie/misterhouse/pull/617.diff

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub #617, or mute the thread https://github.com/notifications/unsubscribe-auth/AExbSNJE6prTHsnJKZRMQLWTkpWQ7dGeks5qrXn0gaJpZM4J_9uk.

@hplato hplato merged commit a2d92bf into hollie:master Nov 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants