-
Notifications
You must be signed in to change notification settings - Fork 78
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
Temper Device with two sensors #19
Comments
You mean the device has both an onboard as well as a cable-bound Can you provide another link to the product? The one you sent merely -- Philipp |
Hi, thank you for your answer :) |
Thanks, that was helpful. It is a total shot in the dark, but if Once installed & rebooted, start it from "C:\Program I have never done that before but in theory the dump should show the -- Philipp |
Hi Philipp, i sent you the files via mail. Thanks again :) |
Thanks, got the files. |
Holy crap, there's an 8-sensor version too: http://www.pcsensor.com/index.php?_a=product&product_id=177 It would be good to come up with an interface which would support polling of 2 or more sensors, maybe something like:
But whoever codes support for this should probably have a 2+ sensor TEMPer to test with. Perhaps we could ask the manufacturers to donate one? |
Hi, But if i get it correct, what you are basically saying, is that the new code shouldn't just support the two sensor version but an "n"-Sensor Version without hard coded stuff for the 2-Sensor version? Thank you for your support so far! |
I've just sent a message to the people at PCSensor asking if they'd be willing to donate one of their 8-thermometer units to our project. I'll update this thread if/when they reply. |
Here's what I got back form PCSensor. They're not willing to donate any hardware 😦...
Nice of them to offer to sell me a DLL for $45 though! 😉 |
My early testing indicates that we can get the external sensor by altering the data_s array references on this line https://github.com/padelt/temper-python/blob/master/temperusb/temper.py#L172 from: I'll play more, and see if I can open a pull request for the functionality. |
Quick update: This makes me more confident |
Hi ps-jay, thank you very much for this change :) I will try it out immediatly. I just added the second sensor to my device to test ;) edit /usr/local/bin/temper-poll |
Yeah - that make sense - that print statement isn't expecting a list of temperatures. Lets see if we can get some action on this project over the next few weeks. @foundbobby has committed some code to select from the cli command switches - we could extend that to support "all", and fix the print statement. |
I originally had it accept all, but saw the error too and removed it. I can add it back if you'd like |
I've update PR #34 ... works for me. |
Updated PR #34 again. |
Just checked #34. Works perfectly with my two sensors: two devices with one sensor each. Thanks to everyone for making an even better solution! Please check if attribution in README.md is correct (what about @foundbobby ?) and send a pull request if not! |
Dear all,
I just bought a Temper-Device with two sensors, one internal and a external (http://www.ebay.de/itm/161240479265?_trksid=p2060778.m2749.l2649&ssPageName=STRK%3AMEBIDX%3AIT)
It is recognized by the system (Bus 001 Device 004: ID 0c45:7401 Microdia) and i can also pull the temperature, but only from one sensor, the internal.
Is it possible to pull the temperature of both sensors?
Thanks!
The text was updated successfully, but these errors were encountered: