Skip to content

Sensor Overflow Logic fix #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.

Sign up for GitHub

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

Merged
merged 7 commits into from
Aug 13, 2018
Merged

Sensor Overflow Logic fix #6

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
e427332
Modified for Issue #5 for the maxCounts to determine if the sensor ov…
Rvice Aug 9, 2018
d970206
Using consts to define the max counts and moved where atime gets calc…
Rvice Aug 9, 2018
9032c5f
Changed logic to >= to capture the case where the sensor reports slig…
Rvice Aug 9, 2018
5e31d3b
Changes to fix linting
Rvice Aug 9, 2018
1566ffd
Moving constants to the same represenation to reflect the datasheet a…
Rvice Aug 10, 2018
22fe497
Added hex comment for reference
Rvice Aug 10, 2018
584a24f
Switch to using the integration time defines
Rvice Aug 10, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 13 additions & 2 deletions adafruit_tsl2591.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,8 @@
_TSL2591_LUX_COEFB = 1.64
_TSL2591_LUX_COEFC = 0.59
_TSL2591_LUX_COEFD = 0.86
_TSL2591_MAX_COUNT_100MS = const(36863) # 0x8FFF
_TSL2591_MAX_COUNT = const(65535) # 0xFFFF

# User-facing constants:
GAIN_LOW = 0x00 # low gain (1x)
Expand Down Expand Up @@ -248,12 +250,21 @@ def lux(self):
and visible light channels.
"""
channel_0, channel_1 = self.raw_luminosity

# Compute the atime in milliseconds
atime = 100.0 * self._integration_time + 100.0

# Set the maximum sensor counts based on the integration time (atime) setting
if self._integration_time == INTEGRATIONTIME_100MS:
max_counts = _TSL2591_MAX_COUNT_100MS
else:
max_counts = _TSL2591_MAX_COUNT

# Handle overflow.
if channel_0 == 0xFFFF or channel_1 == 0xFFFF:
if channel_0 >= max_counts or channel_1 >= max_counts:
raise RuntimeError('Overflow reading light channels!')
# Calculate lux using same equation as Arduino library:
# https://github.com/adafruit/Adafruit_TSL2591_Library/blob/master/Adafruit_TSL2591.cpp
atime = 100.0 * self._integration_time + 100.0
again = 1.0
if self._gain == GAIN_MED:
again = 25.0
Expand Down