Sensor mapping insufficient

[dolfs]

As can be seen from the screenshot, I have a configuration that produces 8 extra sensors. Arguably 7 because #1 is missing. There might be eight easily, however, because I am using the MeteoBridge with a GW1000 from Ecowitt, which in turn supports 8 different channels for additional temp/humidity sensors (there are other types of sensors also supporting up to 8).

The integration's code only checks for extra sensors 1-7 and thus misses my 8th sensor. A comment in another issue suggests that using MeteoBridge sensor mapping may help with this, but it is not clear how. The screenshot was produced without any custom mappings in place. At the least you might want to include documentation to describe which "logical" sensors will be picked up by the integration, so that a suitable one may be chosen.

So I'd like to understand how I can get that 8th sensor to show up, or alternatively for the integration to support 8 extra sensors (7 seems a rather odd number to have been chosen, but that's the engineer in me speaking).

[briis]

Hi,
Unfortunately the current Meteobridge API, has noway of letting me know how many extra sensors are setup on the system. As each sensor adds to the update time, I had to make a decision on how many I would setup - based on user input at that time, I selected 7. But I can add a sensor number 8. I will work on that as soon as I have time.

[dolfs]

You might want to consider, instead of the current slider, a setup UI that offers check boxes for extra sensors. You could basically do 1-8 or 1-16 even. Only the ones with checked boxes would be “read out” to save time if possible. Backwards compatible by taking current slider value and populating 1-<n> initially. Dolf Starreveld — Mountain View, CA www  <https://starreveld.com/>| map  <http://maps.google.com/maps?q=Mountain+View,+CA>| email ***@***.***>

…

On Sep 20, 2022, at 03:09, Bjarne Riis ***@***.***> wrote: Hi, Unfortunately the current Meteobridge API, has noway of letting me know how many extra sensors are setup on the system. As each sensor adds to the update time, I had to make a decision on how many I would setup - based on user input at that time, I selected 7. But I can add a sensor number 8. I will work on that as soon as I have time. — Reply to this email directly, view it on GitHub <#24 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAKFJVI4MDIGHOAU4U63GTLV7GEPLANCNFSM6AAAAAAQPVU27Q>. You are receiving this because you authored the thread.