Migrate MLX90614 sensor driver to new I2C API conventions

[voldemort9999]

While looking at the sensor drivers, I noticed that the MLX90614 module was left in a half-migrated state — it inherits from I2CSlave (which is great), but the rest of the code still followed the old conventions: camelCase methods, print() instead of logging, return False instead of proper None returns, and no type hints or documentation.

Since this was flagged in #182, I went through and did a full cleanup to bring it in line with the rest of the codebase.

Closes #182

Changes

• Renamed methods to snake\_case: getObjectTemperature → get_object_temperature, getAmbientTemperature → get_ambient_temperature, getRaw → get_raw, readReg → read_reg
• Removed getVals — it was a one-line wrapper around self.read(), so I inlined it directly
• Replaced print() with logging.info() in read_reg
• Returns None instead of False on failed reads (more Pythonic, better for type checkers)
• Added type hints to all methods
• Added NumPy-style docstrings to class and all public methods
• Removed commented-out dead code that referenced the old self.I2C.configI2C() API
• init now accepts optional device parameter, consistent with I2CSlave signature

Usage

from pslab.external.MLX90614 import MLX90614

sensor = MLX90614()
print(sensor.get_object_temperature())   # target temp in °C
print(sensor.get_ambient_temperature())  # sensor body temp in °C

Tests

Added tests/test_mlx90614.py with 20 unit tests covering initialization, source selection, temperature conversion math, error handling, logging output, return types, and class attributes. All tests are mock-based and run without hardware.

=================== 20 passed in 0.08s ===================

---

[sourcery-ai]

Sorry @voldemort9999, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

[Copilot]

Pull request overview

Migrates the MLX90614 IR thermometer driver to follow the codebase's modern I2C API conventions (snake_case methods, logging, type hints, NumPy-style docstrings, None instead of False on failure, optional device parameter). Adds a mock-based test module covering initialization, source selection, conversion math, error handling, and logging.

Changes:

• Renamed methods to snake_case, removed the trivial getVals wrapper, replaced print() with logging.info(), and dropped commented-out dead code.
• Added type hints, NumPy-style docstrings, and an optional device: ConnectionHandler constructor parameter consistent with I2CSlave.
• Added tests/test_mlx90614.py with 20 mock-based unit tests.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
pslab/external/MLX90614.py	Full driver cleanup: snake_case API, logging, type hints, docstrings, None-on-failure, optional device param.
tests/test_mlx90614.py	New mock-based unit tests covering init, source selection, conversion, error handling, logging, and return types.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.