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

Provide "Use Cases and Requirements" section #26

Merged
merged 1 commit into from
Apr 28, 2017

Conversation

pozdnyakov
Copy link

@pozdnyakov pozdnyakov commented Apr 28, 2017

Fixes #23

Authors:
Alexander Sahalamov [email protected]
Mikhail Pozdnyakov [email protected]


Preview | Diff

Fixes w3c#23.

Authors:
Alexander Sahalamov <[email protected]>
Mikhail Pozdnyakov <[email protected]>
@anssiko
Copy link
Member

anssiko commented Apr 28, 2017

This is a good start, thanks @pozdnyakov @alexshalamov.

@anssiko anssiko merged commit f95cfcc into w3c:gh-pages Apr 28, 2017
Copy link
Member

@tobie tobie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be useful to include the use cases which only require coarse information, maybe into a separate subsection.

I'm sort of bothered by the highly theoretical nature of some of those use cases. Do we have examples of applications on native platforms doing things like this? I think that would be a lot more convincing. We're not really helping the platform if we're increasing the attack surface area without enabling real use cases.

As mentioned, I think we should focus on getting the right permission solution for motion sensors and this will naturally follow.

health regulations.
1. A Web application monitors light level changes produced by hovering hand user gesture and
interprets them to control a game character.
1. A Web application calculates settings for a camera with manual controls (apperture, shutter speed, ISO).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are the kind of sensors exposed on devices actually useful for this? Are there examples of such apps in native app stores?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So the first one seems to be simply a remote. The second one seems to say it no longer uses ALS because "Ambient light sensor models are unlucky not work" and does pixel analysis of the output of the camera instead.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You asked about apps, I quickly googled, there are apps :D

For high speed photography, I use professional light (laser) sensors / sound triggers. I agree, ALS would not be that useful. For calculating EV values / manual camera settings ALS should be enough for spot metering. Mobile device sensors could measure incident lightning values that are enough to calculate EV values in range of EV -4 to EV 14. I could test readings from my Seconic spot / omni meter and compare to PixelXL readings.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You asked about apps, I quickly googled, there are apps :D

Right… but they don't rely on ALS.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. Those do.

Use Cases and Requirements {#usecases-requirements}
=========

1. A Web application gradually updates document style based on light level changes.
Copy link
Member

@tobie tobie Apr 28, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's typically a CSS domain, and CSS seemed to think that an enum was good enough.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Enum values don't allow to create 'designer' specific ranges or control style changes gradually. ALS sensors on mobile devices could provide values in 0-43000 Lux range. Maybe some pages could provide high-contrast monochrome style for 'direct-sun' conditions. It would be difficult to define 3 enum values for 0-43000 range, that would satisfy everyone.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's not the conclusion at which the CSS WG arrived. :)

Copy link
Member

@tobie tobie Apr 28, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And I think they have more legitimacy in this area than we do.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you happen to have technical details for CSS WG conclusion? Would be nice to sync with it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quite a few references in the mailing list. Seems there are accessibility-related concerns here which seems is why the feature is on hold within the CSS WG.

1. A Web application provides input for a smart home system to control lighting.
1. A Web aplication checks whether light level at work space is sufficient, based on occupational
health regulations.
1. A Web application monitors light level changes produced by hovering hand user gesture and
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have examples of this in shipped software somewhere?

=========

1. A Web application gradually updates document style based on light level changes.
1. A Web application provides input for a smart home system to control lighting.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here, do we have examples of native apps actually doing this?

interprets them to control a game character.
1. A Web application calculates settings for a camera with manual controls (apperture, shutter speed, ISO).

The mentioned use cases require plain light level data, not a pre-defined set of values.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How plain? How accurate? With what frequency requirements? This seems to dependant a lot on the use cases, too.


1. A Web application gradually updates document style based on light level changes.
1. A Web application provides input for a smart home system to control lighting.
1. A Web aplication checks whether light level at work space is sufficient, based on occupational
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are the values of common sensors accurate enough and properly calibrated enough to enable such a use case?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Depends on the country, but usually, office regulations require lightning to provide illumination of 300 Lux, steps between proposed values for different areas (conference rooms, corridors, etc), are 50-100 Lux.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My question was whether onboard sensors were good enough for meaningfully measuring that. My hunch is that onboard sensors are good enough to measure change, but probably not absolute values. I might be completely wrong, however.

Overall, this is a cool use case, I'm just concerned that it is not doable in practice because of HW constraints.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is only one constraint that comes to my mind. Usually, fov for the mobile sensors is limited, that's why, e.g., flash / light meters (that I use, quite often), have omni-directional meter in addition to spot meter. App can fix this limitation, e.g., use sensors + als and ask user to tilt device slightly, to capture als readings for larger fov area.

10 Lux resolution that is guaranteed by sensor manufacturers, should be enough to measure illuminance for workplace.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

10 Lux resolution that is guaranteed by sensor manufacturers, should be enough to measure illuminance for workplace.

Is calibration OK, though? i.e. is 300 Lux measured on a device really 300 ± 10 Lux? That is, resolution is relative, not absolute.

@tobie
Copy link
Member

tobie commented Apr 28, 2017

Authors:
Alexander Sahalamov [email protected]
Mikhail Pozdnyakov [email protected]

As a sidenote, if this is important to you, and I understand there are company cultures in which it is, I'd much rather you added your names to the acknowledgment section than put this kind of stuff in the commit logs.

@anssiko
Copy link
Member

anssiko commented Apr 28, 2017

Updated https://w3c.github.io/ambient-light/#acknowledgements

@lknik
Copy link

lknik commented May 12, 2017

As per Chrome issue - maybe the Web application calculates settings for a camera with manual controls (apperture, shutter speed, ISO) shouldn't be among the most prominent examples?

@anssiko
Copy link
Member

anssiko commented May 12, 2017

@lknik, please provide your feedback on the respective open issue #23 and not on this PR that has been merged.

@tobie
Copy link
Member

tobie commented May 12, 2017

Well, #23 tells you to have this conversation here, so I understand @lknik's confusion. This shouldn't have been merged, should have been reversed or the concerns raised here should have been addressed by the OP.

@anssiko
Copy link
Member

anssiko commented May 12, 2017

Fixed: #23 (comment)

Now let's get back to productive work :-)

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

Successfully merging this pull request may close these issues.

5 participants