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

integration_time equation possibly incorrect #24

Closed
kattni opened this issue May 4, 2020 · 2 comments
Closed

integration_time equation possibly incorrect #24

kattni opened this issue May 4, 2020 · 2 comments

Comments

@kattni
Copy link

kattni commented May 4, 2020

From @kevinjwalters in #16:
FYI, If the RGBC values are of interest, the current defauls set the ADC Integration Time Register (0x81) register to 0x01 which equates to 708.9ms integration time giving updates only ~ 1.4Hz. Page 20 of the datasheet has a slight error on how this value is treated, it says 256 - TIME / 2.78 ms. I'm pretty sure it means (256 - TIME) * 2.78. There's an integration_time property which can be used to adjust this.

I've also noticed clue.color() values getting "stuck" on REPL on the CLUE but I've only seen this once. foamyguy on discord has also seen this.

@fivesixzero
Copy link

I've also noticed some inconsistencies with integration time while doing some testing with a Clue board this week with a logic analyzer in the loop. Timings between bus transactions are sometimes a bit ... off? But that's as far as I got in my "just make sure it works" testing. I'll look into this a bit more closely as I continue down the (quite educational) APDS-9960 rabbit hole and I'll update this issue or submit a PR if I find anything.

As a warning to anyone else working with this sensor: There are (at least?) two different versions of the datasheet floating around. One is from 2013 and the other is from 2015. Putting them side by side, it looks like most or all of the changes between them are in formatting but there could be a few critical typo corrections in there.

@FoamyGuy
Copy link

FoamyGuy commented Jan 8, 2022

I think this is resolved by #39

@FoamyGuy FoamyGuy closed this as completed Jan 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants