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

Fixed problem with netatmo wind guages. #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

andreask
Copy link

Hi,

Thanks for a great plugin, that will solve some of the things I want to do on a project. I have access to a rain module and a wind module. The code you provided for the rain module seems to be working perfectly, however I had to fix the code for the wind module.

2 things:

  1. The information regarding what is present in the wind module is not the same as in the other modules. The information regarding datatype is just "Wind". There are no such DashboardData, so I checked for the module and added the correct values.
  2. In the DashboardData struct, you had a variable called GustStrengthfloat32, which seemed just to be an error. I fixed that as well.

Again, thanks for the work you already put in. By patching this I was able to use it perfectly!

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

Successfully merging this pull request may close these issues.

1 participant