Sunlight Sensor SI1145

[WirthmU]

Could It be possible to add the sunlight sensor SI1145 on I2C to the supported sensors?

[aaronwmorris]

Merged #1401 with SI1145 support. I do not have this sensor, so please let me know if the code works for you.

[WirthmU]

I could configure the sensor as Sensor B on 0x60 on User Slot 15. This is the outcome of the test script:

(indi-allsky) pi@meteotux:~/indi-allsky $ ./misc/sensor_test.py
[WARNING]: Initializing [Sensor A] TSL2561 I2C light sensor device @ 0x29
[WARNING]: Initializing [Sensor B] SI1145 I2C light sensor device @ 0x60
[WARNING]: No sensor C - Initializing sensor simulator
[INFO]: [Sensor A] Switching TSL2561 to day mode - Gain: 0, Integration: 1
[INFO]: [Sensor A] TSL2561 - lux: 17372.3280, broadband: 17914, ir: 6540
[INFO]: Sensor 0: {'sqm_mag': 1.9839143363179694, 'data': (17372.328018047265, 17914, 6540)}
--- Logging error ---
Traceback (most recent call last):
  File "/usr/lib/python3.11/logging/__init__.py", line 1110, in emit
    msg = self.format(record)
          ^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/logging/__init__.py", line 953, in format
    return fmt.format(record)
           ^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/logging/__init__.py", line 687, in format
    record.message = record.getMessage()
                     ^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/logging/__init__.py", line 377, in getMessage
    msg = msg % self.args
          ~~~~^~~~~~~~~~~
TypeError: not enough arguments for format string
Call stack:
  File "/home/pi/indi-allsky/./misc/sensor_test.py", line 182, in <module>
    ts.main()
  File "/home/pi/indi-allsky/./misc/sensor_test.py", line 91, in main
    sensor_data = sensor.update()
  File "/home/pi/indi-allsky/indi_allsky/devices/sensors/lightSensorSi1145.py", line 29, in update
    logger.info('[%s] SI1145 - visible: %d, ir: %d', vis, ir)
Message: '[%s] SI1145 - visible: %d, ir: %d'
Arguments: (5017, 42662)
[INFO]: Sensor 1: {'data': (5017, 42662)}
[INFO]: Sensor 2: {'data': ()}

The MQTT-message generates the following values:
sensor_user_15 = 5017.0
sensor_user_16 = 42662.0

[aaronwmorris]

Merged #1402 to fix this exception. Other than that, it appears to be working.

[WirthmU]

This is what I get now:

(indi-allsky)  pi@meteotux:~/indi-allsky $ ./misc/sensor_test.py
[WARNING]: Initializing [Sensor A] TSL2561 I2C light sensor device @ 0x29
[WARNING]: Initializing [Sensor B] SI1145 I2C light sensor device @ 0x60
[WARNING]: No sensor C - Initializing sensor simulator
[INFO]: [Sensor A] Switching TSL2561 to day mode - Gain: 0, Integration: 1
[ERROR]: SensorReadException: float() argument must be a string or a real number, not 'NoneType'
[INFO]: [Sensor B] SI1145 - visible: 16600, ir: 65535
[INFO]: Sensor 1: {'data': (16600, 65535)}
[INFO]: Sensor 2: {'data': ()}

[aaronwmorris]

That looks better.

As for the exception for the TSL2561. The notes in the library say that a null value is to be expected from time to time. If this were running inside indi-allsky, the previous values for the TSL2561 would not be updated and would just return the old values.

[WirthmU]

The TSL2561 does work, but the SI1145 is stuck since the latest update.

It works now!

Perfect - thank you for your support!

[aaronwmorris]

After reviewing the source code, I think I need to enable the UV index value to add a 3rd data point. The sensor does not have an actual UV sensor, but it can provide an estimate.

[aaronwmorris]

So this was done with the high range mode?

[WirthmU]

The last print screen is with the gain value set to 16 for both, vis and ir.

[aaronwmorris]

Very cool. I am very surprised my code worked on the first try and I do not even have the sensor. I am ordering one for myself so I can finish testing before submitting a PR for the upstream repo.

Does changing the gain seem to affect the readings?

[WirthmU]

Chaning the gain indeed does affect the readings. I did the test in the afternoon with plain sunshine within about 10 minutes:

Gain 1:

(indi-allsky) pi@meteotux:~/indi-allsky/testing $ python SI1145_testing1.py
Visible = 3733, Infrared = 27618
Visible = 3730, Infrared = 27563
Visible = 3733, Infrared = 27545
Visible = 3742, Infrared = 27564
Visible = 3745, Infrared = 27573
Visible = 3744, Infrared = 27584

Gain 2:

(indi-allsky) pi@meteotux:~/indi-allsky/testing $ python SI1145_testing1.py
Visible = 7146, Infrared = 65535
Traceback (most recent call last):
  File "/home/pi/indi-allsky/testing/SI1145_testing1.py", line 26, in <module>
    vis, ir = si1145.als
              ^^^^^^^^^^
  File "/home/pi/indi-allsky/virtualenv/indi-allsky/lib/python3.11/site-packages/adafruit_si1145.py", line 160, in als
    self._send_command(_CMD_ALS_FORCE)
  File "/home/pi/indi-allsky/virtualenv/indi-allsky/lib/python3.11/site-packages/adafruit_si1145.py", line 293, in _send_command
    raise RuntimeError(f"SI1145 Error: {response:#x}")
RuntimeError: SI1145 Error: 0x8d

Gain 4:

(indi-allsky) pi@meteotux:~/indi-allsky/testing $ python SI1145_testing1.py
Visible = 14226, Infrared = 65535
Traceback (most recent call last):
  File "/home/pi/indi-allsky/testing/SI1145_testing1.py", line 26, in <module>
    vis, ir = si1145.als
              ^^^^^^^^^^
  File "/home/pi/indi-allsky/virtualenv/indi-allsky/lib/python3.11/site-packages/adafruit_si1145.py", line 160, in als
    self._send_command(_CMD_ALS_FORCE)
  File "/home/pi/indi-allsky/virtualenv/indi-allsky/lib/python3.11/site-packages/adafruit_si1145.py", line 293, in _send_command
    raise RuntimeError(f"SI1145 Error: {response:#x}")
RuntimeError: SI1145 Error: 0x8d

Gain 8 and above:

(indi-allsky) pi@meteotux:~/indi-allsky/testing $ python SI1145_testing1.py
Visible = 65535, Infrared = 65535
Traceback (most recent call last):
  File "/home/pi/indi-allsky/testing/SI1145_testing1.py", line 26, in <module>
    vis, ir = si1145.als
              ^^^^^^^^^^
  File "/home/pi/indi-allsky/virtualenv/indi-allsky/lib/python3.11/site-packages/adafruit_si1145.py", line 160, in als
    self._send_command(_CMD_ALS_FORCE)
  File "/home/pi/indi-allsky/virtualenv/indi-allsky/lib/python3.11/site-packages/adafruit_si1145.py", line 293, in _send_command
    raise RuntimeError(f"SI1145 Error: {response:#x}")
RuntimeError: SI1145 Error: 0x8d

[aaronwmorris]

My SI1145 sensor should arrive in a few days. I will be able to complete the tests of all of the functions and integrate the day and night gain settings for the sensor after that.

[aaronwmorris]

I am still waiting on my sensor. I am afraid I may have to order a different one. The USPS status says the package will not be delivered due to lack of postage... what a pain.

[WirthmU]

I can order the same as I am using here in Switzerland and send it to you if you like:

[aaronwmorris]

I just put in an order for a different one that is more local to me. Hopefully I will have it in a few days.

[aaronwmorris]

Like I mentioned before, I have already put in an order for another sensor. However, the original sensor I ordered that got held up by USPS also arrived... except it is a VL53L0X time-of-flight sensor :-D I guess I can try to measure the distance to Altair with millimeter accuracy.

[aaronwmorris]

And my second sensor order arrived today. I was able to pretty quickly run all of the testing need to verify all of the properties were working as expected.

Submitted adafruit/Adafruit_CircuitPython_SI1145#21 to the original project. I wanted to get a sanity check from the project before integrating new settings into indi-allsky.

[WirthmU]

Good news, thank you @aaronwmorris.
I am waiting too, because my main PC, which has been down for almost 3 weeks probably because of a faulty PoE card, is not running currently. I have ordered a new system (i.e THE backup) and a replacement card. They call it fitlet-card because it's a fitlet-2 fanless PC.
I will let you know as soon as my system is up and running again.

[aaronwmorris]

I am not going to wait on the upstream. I have simplified the library somewhat to kind of match how some of the other adafruit libraries work. I have updated the python requirements to just use my fork of the adafruit si1145 library for now.

Merged #1489 with the new gain settings for the SI1145.

[WirthmU]

The server is up and running again and I did install the latest version of indi-allsky.

Now I get the notification WARNING: SensorWorker was restarted more than 10 times and all sensor values are 0.

And in the log in the "Tools" section.

2024-09-01T15:40:17.129728+02:00 meteotux [INFO] Capture-1-1324/MainThread capture.saferun() [505]: Image queue depth: 0
2024-09-01T15:40:17.127516+02:00 meteotux [INFO] Capture-1-1324/MainThread libcamera.setCcdExposure() [263]: image command: rpicam-still --immediate --nopreview --camera 0 --raw --denoise off --gain 1 --shutter 314 --metadata /tmp/tmp7ivxt8xi.json --metadata-format json --awbgains 1,1 --output /tmp/tmp58ml4oke.dng
2024-09-01T15:40:17.126579+02:00 meteotux [INFO] Capture-1-1324/MainThread capture.shoot() [1514]: Taking 0.00031430 s exposure (gain 1)
2024-09-01T15:40:16.312232+02:00 meteotux [WARNING] MainProcess-1320/Sensor-6342 lightSensorSi1145.__init__() [96]: Initializing [Sensor B] SI1145 I2C light sensor device @ 0x60
2024-09-01T15:40:15.303937+02:00 meteotux [INFO] MainProcess-1320/MainThread allsky._queueManualTasks() [1210]: Checking for manually submitted tasks
2024-09-01T15:40:15.303700+02:00 meteotux [WARNING] MainProcess-1320/Sensor-6342 lightSensorTsl2561.__init__() [100]: Initializing [Sensor A] TSL2561 I2C light sensor device @ 0x29
2024-09-01T15:40:15.303466+02:00 meteotux [INFO] MainProcess-1320/MainThread allsky._startSensorWorker() [518]: Starting Sensor-6342 worker
2024-09-01T15:40:15.303250+02:00 meteotux [ERROR] MainProcess-1320/MainThread allsky._startSensorWorker() [511]: Sensor worker exception: 
2024-09-01T15:40:15.303030+02:00 meteotux [ERROR] MainProcess-1320/MainThread allsky._startSensorWorker() [511]: Sensor worker exception: AttributeError: module 'adafruit_si1145' has no attribute 'GAIN_ADC_CLOCK_DIV_32'
2024-09-01T15:40:15.302802+02:00 meteotux [ERROR] MainProcess-1320/MainThread allsky._startSensorWorker() [511]: Sensor worker exception:                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-09-01T15:40:15.302586+02:00 meteotux [ERROR] MainProcess-1320/MainThread allsky._startSensorWorker() [511]: Sensor worker exception:     self.vis_gain_night = getattr(adafruit_si1145, self.config.get('TEMP_SENSOR', {}).get('SI1145_VIS_GAIN_NIGHT', 'GAIN_ADC_CLOCK_DIV_32'))
2024-09-01T15:40:15.302366+02:00 meteotux [ERROR] MainProcess-1320/MainThread allsky._startSensorWorker() [511]: Sensor worker exception:   File "/home/pi/indi-allsky/indi_allsky/devices/sensors/lightSensorSi1145.py", line 107, in __init__
2024-09-01T15:40:15.302169+02:00 meteotux [ERROR] MainProcess-1320/MainThread allsky._startSensorWorker() [511]: Sensor worker exception:                       ^^^^^^^^^
2024-09-01T15:40:15.301962+02:00 meteotux [ERROR] MainProcess-1320/MainThread allsky._startSensorWorker() [511]: Sensor worker exception:     self.sensors[1] = b_sensor(
2024-09-01T15:40:15.301769+02:00 meteotux [ERROR] MainProcess-1320/MainThread allsky._startSensorWorker() [511]: Sensor worker exception:   File "/home/pi/indi-allsky/indi_allsky/sensor.py", line 365, in init_sensors
2024-09-01T15:40:15.301600+02:00 meteotux [ERROR] MainProcess-1320/MainThread allsky._startSensorWorker() [511]: Sensor worker exception:     ^^^^^^^^^^^^^^^^^^^
2024-09-01T15:40:15.301382+02:00 meteotux [ERROR] MainProcess-1320/MainThread allsky._startSensorWorker() [511]: Sensor worker exception:     self.init_sensors()  # sensors before dew heater and fan
2024-09-01T15:40:15.301162+02:00 meteotux [ERROR] MainProcess-1320/MainThread allsky._startSensorWorker() [511]: Sensor worker exception:   File "/home/pi/indi-allsky/indi_allsky/sensor.py", line 107, in saferun

And:

(indi-allsky) pi@meteotux:~/indi-allsky $ ./misc/sensor_test.py
[WARNING]: Initializing [Sensor A] TSL2561 I2C light sensor device @ 0x29
[WARNING]: Initializing [Sensor B] SI1145 I2C light sensor device @ 0x60
Traceback (most recent call last):
  File "/home/pi/indi-allsky/./misc/sensor_test.py", line 182, in <module>
    ts.main()
  File "/home/pi/indi-allsky/./misc/sensor_test.py", line 81, in main
    self.init_sensors()
  File "/home/pi/indi-allsky/./misc/sensor_test.py", line 135, in init_sensors
    self.sensors[1] = b_sensor(
                      ^^^^^^^^^
  File "/home/pi/indi-allsky/indi_allsky/devices/sensors/lightSensorSi1145.py", line 107, in __init__
    self.vis_gain_night = getattr(adafruit_si1145, self.config.get('TEMP_SENSOR', {}).get('SI1145_VIS_GAIN_NIGHT', 'GAIN_ADC_CLOCK_DIV_32'))
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: module 'adafruit_si1145' has no attribute 'GAIN_ADC_CLOCK_DIV_32'

[aaronwmorris]

I figured this might be an issue. Because you already have the module installed, it did not upgrade the code.

Try to re-install...

source virtualenv/indi-allsky/bin/activate

pip uninstall adafruit-circuitpython-si1145

pip install git+https://github.com/aaronwmorris/Adafruit_CircuitPython_SI1145.git

[WirthmU]

Looks good:

(indi-allsky) pi@meteotux:~/indi-allsky $ ./misc/sensor_test.py
[WARNING]: Initializing [Sensor A] TSL2561 I2C light sensor device @ 0x29
[WARNING]: Initializing [Sensor B] SI1145 I2C light sensor device @ 0x60
[WARNING]: No sensor C - Initializing sensor simulator
[INFO]: [Sensor A] Switching TSL2561 to night mode - Gain: 1, Integration: 1
[INFO]: [Sensor A] TSL2561 - lux: 0.1206, broadband: 1, ir: 0
[INFO]: Sensor 0: {'sqm_mag': 14.87987678164935, 'data': (0.12063492063492064, 1, 0)}
[INFO]: [Sensor B] Switching SI1145 to night mode - Visible Gain: 5, IR Gain: 5
[INFO]: [Sensor B] SI1145 - visible: 273, ir: 264, uv: 0.000
[INFO]: Sensor 1: {'data': (273, 264, 0.0)}
[INFO]: Sensor 2: {'data': ()}

[aaronwmorris]

Excellent!

[aaronwmorris]

Adafruit has accepted my changes to the SI1145 module. As of #1527, no longer using my fork of the python module.