dsmithson / SpyderTallyController

Tally controller for Spyder video processor using a Raspberry Pi
Apache License 2.0
4 stars 0 forks source link

fix: catching display init error when not available #13

Closed dsmithson closed 1 year ago

dsmithson commented 1 year ago

Resolves a crash on startup that will occur if no display is connected when the Spyder Tallies app starts up. Fix involved adding try/catch logic around the display I2C initialization and adding a null reference check at the start of the updateDisplay method. I was able to verify this issue was occurring with the prior release, and that this fixes the issue.

For good measure I also added similar try/catch handling around the I2C pins used for the actual Tally GPIOs, and I write logs to the console in the case these should fail as well.

fixes #11