QA: Black assisted code quality pass.

Author: Gadgetoid

examples/bargraph.py

@@ -12,15 +12,15 @@
 """
 QNH = 1013.25
 
-BAR_CHAR = u'\u2588'
+BAR_CHAR = "\u2588"
 
-ANSI_COLOR_RED = '\x1b[31m'
-ANSI_COLOR_GREEN = '\x1b[32m'
-ANSI_COLOR_YELLOW = '\x1b[33m'
-ANSI_COLOR_BLUE = '\x1b[34m'
-ANSI_COLOR_MAGENTA = '\x1b[35m'
-ANSI_COLOR_BLACK = '\x1b[30m'
-ANSI_COLOR_RESET = '\x1b[0m'
+ANSI_COLOR_RED = "\x1b[31m"
+ANSI_COLOR_GREEN = "\x1b[32m"
+ANSI_COLOR_YELLOW = "\x1b[33m"
+ANSI_COLOR_BLUE = "\x1b[34m"
+ANSI_COLOR_MAGENTA = "\x1b[35m"
+ANSI_COLOR_BLACK = "\x1b[30m"
+ANSI_COLOR_RESET = "\x1b[0m"
 
 
 colours = [ANSI_COLOR_BLUE, ANSI_COLOR_GREEN, ANSI_COLOR_YELLOW, ANSI_COLOR_RED, ANSI_COLOR_MAGENTA]
@@ -83,12 +83,15 @@ def calculate_altitude(pressure, qnh=1013.25):
         p_bar = p_colour + p_bar + ANSI_COLOR_RESET
         a_bar = a_colour + a_bar + ANSI_COLOR_RESET
 
-        t_reading = "{:.4f}c".format(temperature).ljust(BAR_WIDTH + 14)
-        p_reading = "{:.4f}hPa".format(pressure / 100).ljust(BAR_WIDTH + 14)
-        a_reading = "{:.2f}m".format(altitude).ljust(BAR_WIDTH + 14)
+        t_reading = f"{temperature:.4f}c".ljust(BAR_WIDTH + 14)
+        p_reading = f"{pressure / 100:.4f}hPa".ljust(BAR_WIDTH + 14)
+        a_reading = f"{altitude:.2f}m".ljust(BAR_WIDTH + 14)
 
-        sys.stdout.write('\x1b[0;1H')
-        sys.stdout.write(u"""{title}
+        title = "ICP10125 Sensor".ljust(BAR_WIDTH + 14, " ")
+        blank = " " * (BAR_WIDTH + 14)
+
+        sys.stdout.write("\x1b[0;1H")
+        sys.stdout.write(f"""{title}
 {blank}
 Temperature:  {t_bar}
 {t_reading}
@@ -97,16 +100,7 @@ def calculate_altitude(pressure, qnh=1013.25):
 Altitude:     {a_bar}
 {a_reading}
 {blank}
-""".format(
-            title="ICP10125 Sensor".ljust(BAR_WIDTH + 14, " "),
-            t_bar=t_bar,
-            p_bar=p_bar,
-            a_bar=a_bar,
-            t_reading=t_reading,
-            p_reading=p_reading,
-            a_reading=a_reading,
-            blank=" " * (BAR_WIDTH + 14)
-        ))
+""")
         sys.stdout.flush()
 
 except KeyboardInterrupt:

icp10125/__init__.py

@@ -3,7 +3,7 @@
 
 from smbus2 import SMBus, i2c_msg
 
-__version__ = '0.0.1'
+__version__ = "0.0.1"
 
 NORMAL = 0x6825
 LOW_POWER = 0x609C
@@ -49,7 +49,7 @@ def __init__(self, address=None):
 
         chip_id = self.chip_id()
         if chip_id != CHIP_ID:
-            raise RuntimeError("ICP10125: Invalid Chip ID {:02x}, expected: {:02x}".format(chip_id, CHIP_ID))
+            raise RuntimeError(f"ICP10125: Invalid Chip ID {chip_id:02x}, expected: {CHIP_ID:02x}")
 
         self.read_otp()
 
@@ -72,7 +72,7 @@ def rdwr(self, command, length=0, delay=0):
                 result = list(msg_r)
                 data = []
                 for chunk in range(0, len(result), 3):
-                    if self.crc8(result[chunk:chunk + 2]) != result[chunk + 2]:
+                    if self.crc8(result[chunk : chunk + 2]) != result[chunk + 2]:
                         raise ValueError("ICP10125: Invalid CRC8 in response.")
                     data.append((result[chunk] << 8) | result[chunk + 1])
                 if len(data) == 1:
@@ -84,12 +84,12 @@ def rdwr(self, command, length=0, delay=0):
 
     def chip_id(self):
         result = self.rdwr(READ_ID, 3)
-        return result & 0x3f
+        return result & 0x3F
 
     def read_otp(self):
         move_address_ptr = [
             MOVE_ADDRESS_PTR >> 8, MOVE_ADDRESS_PTR & 0xff,
-            0x00, 0x66, 0x9c  # Address CRC8
+            0x00, 0x66, 0x9C  # Address CRC8
         ]
         self.rdwr(move_address_ptr)
 
@@ -144,7 +144,7 @@ def calculate_conversion_constants(self, p_LUT):
         return A, B, C
 
     def crc8(self, data, polynomial=0x31):
-        result = 0xff
+        result = 0xFF
         for byte in data:
             result ^= byte
             for bit in range(8):
@@ -153,4 +153,4 @@ def crc8(self, data, polynomial=0x31):
                     result ^= polynomial
                 else:
                     result <<= 1
-        return result & 0xff
+        return result & 0xFF

pyproject.toml

@@ -94,6 +94,9 @@ skip = """
 [tool.isort]
 line_length = 200
 
+[tool.black]
+line_length = 200
+
 [tool.check-manifest]
 ignore = [
     '.stickler.yml',

tests/conftest.py

@@ -4,17 +4,17 @@
 import pytest
 
 
-@pytest.fixture(scope='function', autouse=False)
+@pytest.fixture(scope="function", autouse=False)
 def smbus2():
     """Mock smbus module."""
-    sys.modules['smbus2'] = mock.MagicMock()
-    yield sys.modules['smbus2']
-    del sys.modules['smbus2']
+    sys.modules["smbus2"] = mock.MagicMock()
+    yield sys.modules["smbus2"]
+    del sys.modules["smbus2"]
 
 
-@pytest.fixture(scope='function', autouse=False)
+@pytest.fixture(scope="function", autouse=False)
 def icp10125():
     """Import icp10125 module."""
     import icp10125
     yield icp10125
-    del sys.modules['icp10125']
+    del sys.modules["icp10125"]

tests/test_setup.py

@@ -21,4 +21,4 @@ def test_setup_invalid_crc8(smbus2, icp10125):
 
     with pytest.raises(ValueError):
         sensor = icp10125.ICP10125()
-        del sensor
+        del sensor