Skip to content

Add support for PMSA003i - #1501

Merged
OttoWinter merged 22 commits into
esphome:devfrom
sjtrny:pmsa003i
Aug 10, 2021
Merged

Add support for PMSA003i#1501
OttoWinter merged 22 commits into
esphome:devfrom
sjtrny:pmsa003i

Conversation

@sjtrny

@sjtrny sjtrny commented Jan 30, 2021

Copy link
Copy Markdown
Contributor

Description:

Adds support for the PMSA003i sensor, which is an i2c version of similar particulate sensors from Plantower.

Original implementation here https://github.com/sjtrny/pmsa003i

Pull request in esphome-docs with documentation (if applicable):

esphome/esphome.io#973

Checklist:

  • The code change is tested and works locally.
  • Tests have been added to verify that the new code works (under tests/ folder).

If user exposed functionality or configuration variables are added/changed:

@glmnet glmnet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rebase as this would fail marging on dev.

Thanks

Comment thread esphome/components/pmsa003i/pmsa003i.h Outdated
Comment thread esphome/components/pmsa003i/sensor.py Outdated
Comment thread esphome/components/pmsa003i/pmsa003i.cpp Outdated
Comment thread esphome/components/pmsa003i/pmsa003i.cpp Outdated
@sjtrny
sjtrny requested a review from glmnet March 16, 2021 11:05

@OttoWinter OttoWinter left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this contribution! Code looks good to me. Just one minor thing about the struct, can you have a look at it

Comment thread esphome/components/pmsa003i/pmsa003i.h Outdated
Co-authored-by: Otto Winter <otto@otto-winter.com>
@OttoWinter
OttoWinter merged commit 6a2f0f5 into esphome:dev Aug 10, 2021
This was referenced Aug 11, 2021
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants