search

padelt / temper-python

libusb/PyUSB-based driver to read TEMPer USB HID devices (USB ID 0c45:7401) and serve as a NetSNMP passpersist module
Other
182 stars 76 forks source link

Sorry, not an issue - adding V1.1 support #120

Closed bjornenki closed 2 years ago

bjornenki commented 2 years ago

Hi, thanks for this project!

Sorry it's not a pull request, but in case it helps others, I just wanted to say that I have some old V1.1 units that work just fine with the following in device_library.py:

"TEMPerHumiV1.1": TemperConfig(
    temp_sens_offsets=[2],
    hum_sens_offsets=[4],
    type=TemperType.FM75,
),

Sample output:

Found 2 devices Device #0: 25.7°C 78.2°F 46.2%RH Device #1: 25.6°C 78.0°F 46.1%RH

davet2001 commented 2 years ago

Many thanks for your contribution! If no one else gets to it first, I will create a PR in a few days time 👍

davet2001 commented 2 years ago

Sample output:

Found 2 devices Device #0: 25.7°C 78.2°F 46.2%RH Device #1: 25.6°C 78.0°F 46.1%RH

@bjornenki Did you have 2 devices plugged in at the time, or does the one USB device have two T and two RH sensors?

bjornenki commented 2 years ago

Yes, sorry for the confusion, but I did have two different 1.0 devices plugged in at once.

On Mon, Apr 18, 2022 at 6:07 AM Dave T @.***> wrote:

Sample output:

Found 2 devices Device #0: 25.7°C 78.2°F 46.2%RH Device #1 https://github.com/padelt/temper-python/pull/1: 25.6°C 78.0°F 46.1%RH

@bjornenki https://github.com/bjornenki Did you have 2 devices plugged in at the time, or does the one USB device have two T and two RH sensors?

— Reply to this email directly, view it on GitHub https://github.com/padelt/temper-python/issues/120#issuecomment-1101393551, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA7Y32ZJXR2MEE56WDK4SLVFVM63ANCNFSM5SNNL6OQ . You are receiving this because you were mentioned.Message ID: @.***>

davet2001 commented 2 years ago

Now implemented on master. Thanks @bjornenki !