sparkfun / apds-9301_breakout Goto Github PK

1. Are measurements included for outer dimension and standoffs (Eagle layer 47)? [NO]
   comment: Measurement units are not included.

2. Are all PTHs (aka connectors) clearly labeled with their purpose on both sides of the board? [NO]
   comment: Small form factor, no space for silk on underside.

Since you have the space could you beef up the size of the address and VCC range text on the schematic?

Thanks,
Elias

After reading light levels from multiple APDS-9301 boards using this library, I have found that the values returned by APDS9301::readLuxLevel() are at least an order of magnitude lower than expected and as measured from a phone app (though such measurements are not expected to be terribly accurate). Still, I found this library differs from how the TSL2561 library treats the empirical formula for lux found in the APDS-9301 data sheet. This library assumes that formula applies to the low gain readings, while the latter assumes it applies to the high gain readings (see here); the data sheet itself is ambiguous as to the gain to be used in that formula.

The end result is that the two libraries give lux values different by a factor of 16. Given my comparison with other sensors, I suspect the scaling in this library is incorrect. A fix would be to replace:

  if (getGain() == HIGH_GAIN)
  {
    ch0 /= 16;
    ch1 /= 16;
  }

with:

  if (getGain() == LOW_GAIN) 
  {
    ch0 *= 16;
    ch1 *= 16;
  }

Can anyone confirm a scaling issue and, if so, apply the fix?