Skip to content
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

Ambient Light Sensors to detect RGB colors #9

Open
lknik opened this issue Apr 13, 2016 · 11 comments
Open

Ambient Light Sensors to detect RGB colors #9

lknik opened this issue Apr 13, 2016 · 11 comments
Milestone

Comments

@lknik
Copy link

lknik commented Apr 13, 2016

Modern sensors are capable of detecting not only luminance, but also actual RGB colors (e.g. http://ams.com/eng/Products/Light-Sensors/Color-Sensors-Proximity-Detection/TMD37821).

Might be interesting to evaluate the feasibility of including it to the spec/draft at some point. Note: it appears there are no actual devices currently supporting this.

Also, in case this ships, it should be directly subject to permissions.

@tobie
Copy link
Member

tobie commented Apr 14, 2016

Thanks.

As discussed, until this is exposed by underlying platforms and shipped in devices there is little point in looking at it more seriously.

I'll keep this open for now in case someone has more on this topic, else we'll revisit whenthese actually show up in devices.

@alexshalamov
Copy link

@anssiko Some Android devices as well as Windows API provide temperature in addition to 'clear channel' info. Temperature can be converted to RGB if needed. Do you think it worth experimenting with implementing support for it? RGB data can be used in e.g., accessibility applications or provided as an input for WB to Image Capture API.

@anssiko
Copy link
Member

anssiko commented Aug 21, 2017

I'm all in for experimentation. Given good use cases and developer excitement we should consider baking it into a future spec update.

(To clarify, we're talking about measuring color temperature, not infrared radiation.)

@anssiko anssiko added this to the Level 2 milestone Oct 5, 2017
@anssiko
Copy link
Member

anssiko commented Oct 5, 2017

I added this issue to the Level 2 milestone.

@reillyeon
Copy link
Member

Discussed in F2F at TPAC 2018. Related feature request to immersive-web/proposals#27. @AdaRoseCannon @kyungtae

@anssiko
Copy link
Member

anssiko commented Oct 23, 2018

@rakuco
Copy link
Member

rakuco commented Dec 14, 2018

For the record since the immersive-web link above no longer works: the issue above was moved and is now immersive-web/lighting-estimation#1

@anssiko
Copy link
Member

anssiko commented Jan 15, 2019

immersive-web/lighting-estimation#1 seems to pursue an XR Device API-specific ALS extension that is tied with an XRFrame. The ALS API's proposed RGB feature might still be useful for a webxr-polyfill. Cc @AdaRoseCannon to comment whether that'd be a valid use case.

Another use case described in this issue is RGB data as an input to white balance in conjunction with Image Capture API. Cc @riju who has implemented some Image Capture API enhancements and experimented with OpenCV.js to investigate.

@AdaRoseCannon
Copy link

A polyfill would definitely be a valid use case.

@anssiko
Copy link
Member

anssiko commented Jan 16, 2019

Thank you @AdaRoseCannon. We will loop you in for review when this feature work moves forward to ensure it addresses your polyfill requirements.

@rzr
Copy link

rzr commented Feb 28, 2019

Well, I also observed that there is no color sensor in generic sensors spec,
so I made a basic implementation of tcs34725 for IoT.js (it runs on node as well)

https://github.com/samsunginternet/color-sensor-js

It's very basic, and schema is just :

{
color: "#RrGgBb" 
}

Merging in Ambient light could makes sense, but in the case of tcs34725 It could be misleading because this sensor is active: it ships a white LED that light the material to observe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants