Skip to content

Add support for Bresser CO2 PN 7009977 and HCHO/VOC PN 7009978 sensors #2815

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

Merged
merged 6 commits into from
Feb 3, 2024

Conversation

ProfBoc75
Copy link
Collaborator

This pr is related to the 2 issues #2813 #2814 in order to add support for the 2 Air quality sensors

CO2 sensor PN 7009977
HCHO / VOC sensor PN 7009978

These sensors are based on the same protocol as current Bresser 7in1 weather station. This PR is updating this device.

Thx.


/**
Decoder for Bresser Weather Center 7-in-1 and Air Quality PM2.5 / PM10, outdoor sensors.
Decoder for Bresser Weather Center 7-in-1 and Air quality sensors.
Air Quality PM2.5/PM10 PN 7009970
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is markdown, for nice display the three device lines should start with a dash - Air…

Raw :
SType Startup & Channel
| |
{207}dab6d782acd9 a 1 ad9aad9aad9aaaaaaaaaaaaaaaaae99aaaaa00 Type = 0xa = 10, Startup = 0, ch = 1
Copy link
Collaborator

Choose a reason for hiding this comment

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

Code blocks need to be indented 4 spaces (preview in any markdown editor)

/* clang-format off */
data = data_make(
"model", "", DATA_STRING, "Bresser-7in1",
"model", "", DATA_STRING, "Bresser-AIR-PM",
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can add // should be "Bresser-Air" but you can not change existing model names.

@zuckschwerdt
Copy link
Collaborator

Thank you so much for working through analysis and implementation so quickly!

@ProfBoc75
Copy link
Collaborator Author

Thank you so much for working through analysis and implementation so quickly!

Thanks, with pleasure. Most of the decode part was already in place, I "just" added 2 sensor types. 😉

@Mart124
Copy link

Mart124 commented Feb 3, 2024

Many thanks @ProfBoc75 for having handled this, really appreciated 👍
And thank you for your review @zuckschwerdt !
Did you perhaps plan to buy these sensors @ProfBoc75 ? :)
I'll of course give this enhancement a try as soon as the PR will be merged (as my Home Assistant add-on is based on the rtl_433 master branch).
Thank you again !

@zuckschwerdt zuckschwerdt merged commit e676082 into merbanan:master Feb 3, 2024
@ProfBoc75 ProfBoc75 deleted the patch-1 branch February 3, 2024 13:58
@Mart124
Copy link

Mart124 commented Feb 3, 2024

I confirm both devices are now properly supported, thank you very much again 👍
Really glad :)

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.

New device Bresser HCHO/VOC sensor, PN 7009978 New device Bresser CO2 sensor, PN 7009977
3 participants