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

Faulty readings #11

Open
tony1tf opened this issue Aug 19, 2024 · 1 comment
Open

Faulty readings #11

tony1tf opened this issue Aug 19, 2024 · 1 comment

Comments

@tony1tf
Copy link

tony1tf commented Aug 19, 2024

I've been running Python code with your library (from breakout_as7343 import BreakoutAS7343) to show the colour channels as bar graphs. However, 3 channels show low readings, which are not analogue. With low light level on channels 515nm, 600nm and 690nm, the output from the sensor is 11.0. With a higher light level the three channels jump to 139.0. Other working channels show readings in the 1000's. Is this a problem with your driver code, or do I have a faulty sensor?

@Gadgetoid
Copy link
Member

I think I made a faulty assumption about the location of "ASTATUS" output in the FIFO, which has - as a consequence - shifted all the readings over by one position.

Because F1, F2 and FZ are the first readings from each result cycle they get replaced with the status data and each reading in turn gets replaced with its adjacent reading. I have no earthly clue how this wasn't completely and totally obvious to me when I wrote the two drivers.

In addition to this, result cycle 3 has been amended in the datasheet from "F1, F5, F7, F8, 2xVIS, FD" to "F1, F7, F8, F5, 2xVIS, FD", I've no idea when this happened because the datasheet neglects to clarify (for the record, it does say that these values were changed) but that would mean F5, F7 and F8 would be wrong even if I hadn't mucked up the status stuff.

AIUI now the FIFO output for each cycle is: Status, CH1, CH2, CH3, CH4, CH5, CH6, Flicker Detect.

Gadgetoid added a commit that referenced this issue Sep 5, 2024
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