code style fixed

.github/workflows/docs.yml

@@ -25,6 +25,7 @@ jobs:
           python ./docs/source/generate_class_list.py
       - name: Build Docs
         run: |
+          cp LICENSE docs/source/
           cd docs
           sphinx-apidoc -o ./source/ ../src/ -M --no-toc
           rm ./source/*.rst

docs/source/generate_class_list.py

@@ -11,8 +11,10 @@ def write_classes(f: TextIOWrapper, patterns: list[str], module_name: str, title
     classes = [
         name for name, obj in inspect.getmembers(module, inspect.isclass)
         if (obj.__module__ == module_name and
-            any(fnmatch.fnmatch(name, pat) for pat in patterns if pat not in exclude) and
-            obj.__doc__ and '(Automatic generated stub)' not in obj.__doc__)
+            any(fnmatch.fnmatch(name, pat) for pat in patterns) and
+            obj.__doc__ and
+            # '(Automatic generated stub)' not in obj.__doc__ and
+            '(no I/O function)' not in obj.__doc__)
     ]
 
     """Write the classes to the file."""
@@ -21,23 +23,26 @@ def write_classes(f: TextIOWrapper, patterns: list[str], module_name: str, title
         f.write(f'{description}\n\n')
 
     for cls in classes:
-        f.write('```{eval-rst}\n')
-        f.write(f'.. autoclass:: {module_name}.{cls}\n')
-        f.write('   :members:\n')
-        f.write('   :undoc-members:\n')
-        f.write('   :show-inheritance:\n')
-        f.write('   :inherited-members:\n')
-        if title != 'Base classes':
-            f.write('   :exclude-members: select\n')
-        f.write('```\n\n')
+        if cls not in exclude:
+            f.write('```{eval-rst}\n')
+            f.write(f'.. autoclass:: {module_name}.{cls}\n')
+            f.write('   :members:\n')
+            f.write('   :show-inheritance:\n')
+            f.write('   :inherited-members: object\n')
+            if title not in ['Base classes', 'Bus coupler']:
+                f.write('   :exclude-members: select, parameters\n')
+            else:
+                print('* ', cls)
+            f.write('```\n\n')
 
 
 with open('docs/source/modules.md', 'w') as f:
-    f.write('# Classes\n\n')
+    f.write('# Classes and Modules\n\n')
     write_classes(f, ['BK*', 'WAGO_750_352'], 'pyhoff.devices', title='Bus coupler',
                   description='These classes are bus couplers and are used to connect the IO bus terminals to a Ethernet interface.')
     write_classes(f, ['KL*'], 'pyhoff.devices', title='Beckhoff bus terminals')
     write_classes(f, ['WAGO*'], 'pyhoff.devices', title='WAGO bus terminals', exclude=['WAGO_750_352'])
+    write_classes(f, ['*Terminal*'], 'pyhoff.devices', title='Generic bus terminals')
     write_classes(f, ['*'], 'pyhoff', title='Base classes',
                   description='These classes are base classes for devices and are typically not used directly.')
     write_classes(f, ['*'], 'pyhoff.modbus', title='Modbus',

src/pyhoff/devices.py

@@ -224,7 +224,7 @@ class KL3054(AnalogInputTerminal):
             channel: The channel number to read from.
 
         Returns:
-            The current value.
+            The current value in mA.
         """
         return self.read_normalized(channel) * 16.0 + 4.0
 
@@ -244,7 +244,7 @@ class KL3042(AnalogInputTerminal):
             channel: The channel number to read from.
 
         Returns:
-            The current value.
+            The current value in mA.
         """
         return self.read_normalized(channel) * 20.0
 
@@ -311,7 +311,7 @@ class KL4002(AnalogOutputTerminal):
 
         Args:
             channel: The channel number to set.
-            value: The voltage value to set.
+            value: The voltage value to set in V.
 
         Returns:
             True if the write operation succeeded.
@@ -348,7 +348,7 @@ class KL4132(AnalogOutputTerminal):
 
         Args:
             channel: The channel number to set.
-            value: The voltage value to set.
+            value: The voltage value to set in V.
 
         Returns:
             True if the write operation succeeded.
@@ -369,7 +369,7 @@ class KL4004(AnalogOutputTerminal):
 
         Args:
             channel: The channel number to set.
-            value: The voltage value to set.
+            value: The voltage value to set in V.
 
         Returns:
             True if the write operation succeeded.