Skip to content

Fix dht22 when no data was read initially #8976#9198

Merged
andrey-git merged 1 commit into
home-assistant:devfrom
maweki:dhtfix
Aug 28, 2017
Merged

Fix dht22 when no data was read initially #8976#9198
andrey-git merged 1 commit into
home-assistant:devfrom
maweki:dhtfix

Conversation

@maweki
Copy link
Copy Markdown
Contributor

@maweki maweki commented Aug 28, 2017

This fixes #8976
When no data was available the module crashes.

Description:

Related issue (if applicable): fixes #8976

This fixes home-assistant#8976
When no data was available the module crashes.
@homeassistant
Copy link
Copy Markdown
Contributor

Hi @maweki,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

@maweki
Copy link
Copy Markdown
Contributor Author

maweki commented Aug 28, 2017

I don't know whether we would want to write a debug statement on the else path. I didn't but I can add that.

@andrey-git
Copy link
Copy Markdown
Contributor

Thanks!

@andrey-git andrey-git merged commit e76e9e0 into home-assistant:dev Aug 28, 2017
@andrey-git andrey-git mentioned this pull request Aug 28, 2017
matemaciek added a commit to matemaciek/home-assistant that referenced this pull request Aug 30, 2017
* upstream/dev: (113 commits)
  Fix fitbit error when trying to access token after upgrade. (home-assistant#9183)
  Upgrade sendgrid to 5.0.1 (home-assistant#9215)
  Upgrade pyasn1 to 0.3.3 and pyasn1-modules to 0.1.1 (home-assistant#9216)
  directv: extended discovery via REST api, bug fix  (home-assistant#8800)
  Bayesian Binary Sensor (home-assistant#8810)
  Add cloud auth support (home-assistant#9208)
  Abode push events and lock, cover, and switch components (home-assistant#9095)
  Lint Sonarr tests
  Upgrade pymysensors to 0.11.1 (home-assistant#9212)
  Refactor rfxtrx (home-assistant#9117)
  Issue home-assistant#6893 in rfxtrx (home-assistant#9130)
  Support for season sensor (home-assistant#8958)
  Add counter component (home-assistant#9146)
  Fix and optimize digitalloggers platform (home-assistant#9203)
  Prevent error when no forecast data was available (home-assistant#9176)
  Add "status" to Sonarr sensor (home-assistant#9204)
  fix worldtidesinfo home-assistant#9184 (home-assistant#9201)
  Update pushbullet.py (home-assistant#9200)
  Fix dht22 when no data was read initially home-assistant#8976 (home-assistant#9198)
  Prevent iCloud exceptions in logfile (home-assistant#9179)
  ...
@balloob balloob mentioned this pull request Sep 7, 2017
@home-assistant home-assistant locked and limited conversation to collaborators Dec 11, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DHT22 errors after 0.51

4 participants