-
Notifications
You must be signed in to change notification settings - Fork 586
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
API support for SenseHat V2 Color Sensor TCS34725 #2088
Comments
Feel free to raise a PR to add it. We're not tracking the binding requests. So I'm leaving this one open a little bit but it will be closed in one of the next triage we'll collectively have. |
[Triage] You'll need to create something similar to https://github.com/dotnet/iot/blob/main/src/devices/SenseHat/SenseHatMagnetometer.cs and add property like this: https://github.com/dotnet/iot/blob/main/src/devices/SenseHat/SenseHat.cs#L48 - you need to find address first and handle errors in case someone is still using older version of SenseHat |
So while investigating, I found out that I was wrong about the sensor. The color sensor on the SenseHat seems to have changed recently to a different model: TSC3400 (which is not covered in the device bindings as of yet). Source: astro-pi/python-sense-hat#126 (comment) I'm sorry about that, but my low level skills aren't good enough to implement the other sensor. :) |
@Ducki All what we can do is leaving the ticket open for a while to see if there is someone owning the hardware and willing to contribute with a PR. If this doesn't happen, the issue will be closed. |
I just checked and indeed, the sensors are different. The excellent news is that they are vey close. So it should be a matter of hours to adjust/adapt/extend the existing one to support the 3400 ones. 3400 seems to have Infra red as well. I don't have this sensor at all but I'm happy to support and help. |
The revision 2 of the Raspberry Pi SenseHat has a color sensor TCS34725, which is currently unsupported in the SenseHat bindings.
The sensor itself is already implemented, but it would be nice, if it was usable via the SenseHat binding.
The text was updated successfully, but these errors were encountered: