Skip to content

Parameter settings of integration time may be wrong. #6

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

Closed
ssagat opened this issue Oct 25, 2019 · 1 comment
Closed

Parameter settings of integration time may be wrong. #6

ssagat opened this issue Oct 25, 2019 · 1 comment
Assignees

Comments

@ssagat
Copy link

ssagat commented Oct 25, 2019

From "Designing the VEML7700 Into an Application"(https://www.vishay.com/docs/84323/designingveml7700.pdf),

REGISTER NAME BIT FUNCTION / DESCRIPTION R / W
ALS_IT 9 : 6
ALS integration time setting
1100 = 25 ms
1000 = 50 ms
0000 = 100 ms
0001 = 200 ms
0010 = 400 ms
0011 = 800 ms

So, I think the parameters of integration time settings should be,
ALS_25MS = const(0xC)
ALS_50MS = const(0x8)
ALS_100MS = const(0x0)
ALS_200MS = const(0x1)
ALS_400MS = const(0x2)
ALS_800MS = const(0x3)

Because of the current wrong parameter settings, both "light" and "lux" values are different from other sample codes.

siddacious added a commit to siddacious/Adafruit_CircuitPython_VEML7700 that referenced this issue Oct 25, 2019
ladyada added a commit that referenced this issue Oct 25, 2019
fixing ALS Integration time consts for #6
@siddacious
Copy link
Contributor

Thanks @ssagat you were indeed correct. The datasheet isn't the clearest!

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

2 participants