search

matthias-bs / BresserWeatherSensorReceiver

Bresser 5-in-1/6-in-1/7-in-1 868 MHz Weather Sensor Radio Receiver for Arduino based on CC1101, SX1276/RFM95W or SX1262
MIT License
121 stars 23 forks source link

Do I need to uncomment SENSOR_TYPE_AIR_PM? #186

Closed simonsh closed 4 months ago

simonsh commented 4 months ago

Do I need to uncomment SENSOR_TYPE_AIR_PM anywhere. The sensor arrives next week, thanks!

matthias-bs commented 4 months ago

No, non-existing sensors won't cause any problems.

simonsh commented 4 months ago

It will exist when I set it up next week.

matthias-bs commented 4 months ago

I expect it will work, because I am using it, too.

simonsh commented 4 months ago

OK, I now have a 5-in-1 weather station (new version), lightning sensor and PM AQ sensor. All working and your code is picking them up perfectly. I uncommented 7-in-1 to receive the AQ sensor, though it is available separately, so you might want to revisit that aspect of the config. to allow it it to be set independently.

When I have made some more progress with my improved 7" ESP-S3 display, I will post a photo. Good work!

matthias-bs commented 4 months ago

That's great!

matthias-bs commented 4 months ago

You are right, the code size could be reduced by enabling/disabling parts of the 7-in-1 decoder. On the other hand, I don't want to make the configuration even more complicated than it already is.