Merge pull request #21 from orionrobots/enable-test-workflows

dannystaple · web-flow · commit 2b420afbbdd7 · 2025-01-25T23:06:20.000Z
Fix the test workflows #8
diff --git a/.editorconfig b/.editorconfig
@@ -11,3 +11,6 @@ charset = utf-8
 trim_trailing_whitespace = true
 insert_final_newline = true
 max_line_length = 120
+
+[*.yml, *.yaml]
+indent_size = 2
diff --git a/.github/workflows/python-test.yml b/.github/workflows/python-test.yml
@@ -1,40 +1,40 @@
 # This workflow will install Python dependencies, run tests and lint with a variety of Python versions
 # For more information see: https://help.github.com/actions/language-and-framework-guides/using-python-with-github-actions
-
+---
 name: Python package
 
 on:
   push:
-    branches: [ master ]
+    branches: [ main ]
   pull_request:
-    branches: [ master ]
+    branches: [ main ]
 
 jobs:
   test:
 
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: [3.7, 3.8, 3.9]
+        python-version: [3.9, 3.11, 3.13]
 
     steps:
-    - uses: actions/checkout@v2
-    - name: Set up Python ${{ matrix.python-version }}
-      uses: actions/setup-python@v2
-      with:
-        python-version: ${{ matrix.python-version }}
-    - name: Install dependencies
-      run: |
-        python -m pip install --upgrade pip
-        python -m pip install flake8 pytest
-        python -m pip install -e .
-#        if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
-    - name: Lint with flake8
-      run: |
-        # stop the build if there are Python syntax errors or undefined names
-        flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
-        # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
-        flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
-#     - name: Test with pytest
-#       run: |
-#         pytest
+      - uses: actions/checkout@v2
+      - name: Set up Python ${{ matrix.python-version }}
+        uses: actions/setup-python@v2
+        with:
+            python-version: ${{ matrix.python-version }}
+      - name: Install dependencies
+        run: |
+            python -m pip install --upgrade pip
+            python -m pip install flake8 pytest
+            python -m pip install -e .
+    #        if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
+      - name: Lint with flake8
+        run: |
+            # stop the build if there are Python syntax errors or undefined names
+            flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
+            # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
+            flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
+    #     - name: Test with pytest
+    #       run: |
+    #         pytest
diff --git a/README.md b/README.md
@@ -30,7 +30,7 @@ This has previously been tested on Linux, OSX and Windows. OSX and Windows requi
 
 As a library:
 
-    >>> import usb_arm
+    >>> import owi_maplin_usb_arm as usb_arm
 
 To initialise libusb and the arm
 
diff --git a/demos/bluedot/led_only.py b/demos/bluedot/led_only.py
@@ -1,16 +1,16 @@
 from bluedot import BlueDot
 from signal import pause
-import usb_arm
+import owi_maplin_robot_arm
 
-arm = usb_arm.Arm()
+arm = owi_maplin_robot_arm.Arm()
 
 
 def led_pressed(pos):
-    arm.tell(usb_arm.LedOn)
+    arm.tell(owi_maplin_robot_arm.LedOn)
 
 
 def stop(*args):
-    arm.tell(usb_arm.Stop)
+    arm.tell(owi_maplin_robot_arm.Stop)
 
 
 bd = BlueDot(cols=3, rows=2)
diff --git a/demos/bluedot/simple_grid.py b/demos/bluedot/simple_grid.py
@@ -1,51 +1,51 @@
 from bluedot import BlueDot
 from signal import pause
-import usb_arm
+import owi_maplin_robot_arm
 
-arm = usb_arm.Arm()
+arm = owi_maplin_robot_arm.Arm()
 
 
 def led_pressed(pos):
-    arm.tell(usb_arm.LedOn)
+    arm.tell(owi_maplin_robot_arm.LedOn)
 
 
 def stop(*args):
-    arm.tell(usb_arm.Stop)
+    arm.tell(owi_maplin_robot_arm.Stop)
 
 
 def grip_pressed(pos):
     if pos.top:
-        arm.tell(usb_arm.GripsClose)
+        arm.tell(owi_maplin_robot_arm.GripsClose)
     if pos.bottom:
-        arm.tell(usb_arm.GripsOpen)
+        arm.tell(owi_maplin_robot_arm.GripsOpen)
 
 
 def wrist_pressed(pos):
     if pos.top:
-        arm.tell(usb_arm.WristUp)
+        arm.tell(owi_maplin_robot_arm.WristUp)
     if pos.bottom:
-        arm.tell(usb_arm.WristDown)
+        arm.tell(owi_maplin_robot_arm.WristDown)
 
 
 def elbow_pressed(pos):
     if pos.top:
-        arm.tell(usb_arm.ElbowUp)
+        arm.tell(owi_maplin_robot_arm.ElbowUp)
     if pos.bottom:
-        arm.tell(usb_arm.ElbowDown)
+        arm.tell(owi_maplin_robot_arm.ElbowDown)
 
 
 def shoulder_pressed(pos):
     if pos.top:
-        arm.tell(usb_arm.ShoulderUp)
+        arm.tell(owi_maplin_robot_arm.ShoulderUp)
     if pos.bottom:
-        arm.tell(usb_arm.ShoulderDown)
+        arm.tell(owi_maplin_robot_arm.ShoulderDown)
 
 
 def base_pressed(pos):
     if pos.left:
-        arm.tell(usb_arm.BaseCtrClockWise)
+        arm.tell(owi_maplin_robot_arm.BaseCtrClockWise)
     if pos.right:
-        arm.tell(usb_arm.BaseClockWise)
+        arm.tell(owi_maplin_robot_arm.BaseClockWise)
 
 
 bd = BlueDot(cols=3, rows=2)
diff --git a/demos/key_ctrl.py b/demos/key_ctrl.py
@@ -1,6 +1,6 @@
 """key_ctrl - key based arm controller"""
 from functools import partial
-import usb_arm
+import owi_maplin_robot_arm
 import pygame
 from pygame.locals import *
 import sys
@@ -18,22 +18,22 @@ def make_keymap():
     """Bp - an initialised arm bitpattern.
     returns the keymap"""
     return {
-        K_z: usb_arm.BaseClockWise,
-        K_x: usb_arm.BaseCtrClockWise,
-        K_r: usb_arm.CloseGrips,
-        K_f: usb_arm.OpenGrips,
-        K_a: usb_arm.ShoulderDown,
-        K_q: usb_arm.ShoulderUp,
-        K_s: usb_arm.ElbowDown,
-        K_w: usb_arm.ElbowUp,
-        K_d: usb_arm.WristDown,
-        K_e: usb_arm.WristUp}
+        K_z: owi_maplin_robot_arm.BaseClockWise,
+        K_x: owi_maplin_robot_arm.BaseCtrClockWise,
+        K_r: owi_maplin_robot_arm.CloseGrips,
+        K_f: owi_maplin_robot_arm.OpenGrips,
+        K_a: owi_maplin_robot_arm.ShoulderDown,
+        K_q: owi_maplin_robot_arm.ShoulderUp,
+        K_s: owi_maplin_robot_arm.ElbowDown,
+        K_w: owi_maplin_robot_arm.ElbowUp,
+        K_d: owi_maplin_robot_arm.WristDown,
+        K_e: owi_maplin_robot_arm.WristUp}
 
 
 def key_loop():
     km = make_keymap()
     try:
-        arm = usb_arm.Arm()
+        arm = owi_maplin_robot_arm.Arm()
     except AttributeError:
         print("Please make sure the arm is connected and turned on")
         sys.exit(1)
diff --git a/demos/web_arm/arm_hello.py b/demos/web_arm/arm_hello.py
@@ -1,16 +1,16 @@
-import usb_arm
+import owi_maplin_robot_arm
 from flask import Flask, Response
 
 
-arm = usb_arm.Arm()
+arm = owi_maplin_robot_arm.Arm()
 app = Flask(__name__)
 
 
 @app.route('/')
 def flash():
     def inner():
         yield "starting</br>\n"
-        arm.move(usb_arm.LedOn)
+        arm.move(owi_maplin_robot_arm.LedOn)
         yield "completed\n"
     return Response(inner())
 
diff --git a/demos/web_arm/arm_server.py b/demos/web_arm/arm_server.py
@@ -1,25 +1,25 @@
-import usb_arm
+import owi_maplin_robot_arm
 from flask import Flask, Response, request
 
 
-arm = usb_arm.Arm()
+arm = owi_maplin_robot_arm.Arm()
 app = Flask(__name__)
 
 movemap = {
-    "GripsClose": usb_arm.GripsClose,
-    "CloseGrips": usb_arm.CloseGrips,
-    "GripsOpen": usb_arm.GripsOpen,
-    "OpenGrips": usb_arm.OpenGrips,
-    "Stop": usb_arm.Stop,
-    "WristUp": usb_arm.WristUp,
-    "WristDown": usb_arm.WristDown,
-    "ElbowUp": usb_arm.ElbowUp,
-    "ElbowDown": usb_arm.ElbowDown,
-    "ShoulderUp": usb_arm.ShoulderUp,
-    "ShoulderDown": usb_arm.ShoulderDown,
-    "BaseClockWise": usb_arm.BaseClockWise,
-    "BaseCtrClockWise": usb_arm.BaseCtrClockWise,
-    "LedOn": usb_arm.LedOn
+    "GripsClose": owi_maplin_robot_arm.GripsClose,
+    "CloseGrips": owi_maplin_robot_arm.CloseGrips,
+    "GripsOpen": owi_maplin_robot_arm.GripsOpen,
+    "OpenGrips": owi_maplin_robot_arm.OpenGrips,
+    "Stop": owi_maplin_robot_arm.Stop,
+    "WristUp": owi_maplin_robot_arm.WristUp,
+    "WristDown": owi_maplin_robot_arm.WristDown,
+    "ElbowUp": owi_maplin_robot_arm.ElbowUp,
+    "ElbowDown": owi_maplin_robot_arm.ElbowDown,
+    "ShoulderUp": owi_maplin_robot_arm.ShoulderUp,
+    "ShoulderDown": owi_maplin_robot_arm.ShoulderDown,
+    "BaseClockWise": owi_maplin_robot_arm.BaseClockWise,
+    "BaseCtrClockWise": owi_maplin_robot_arm.BaseCtrClockWise,
+    "LedOn": owi_maplin_robot_arm.LedOn
 }
 
 
diff --git a/demos/web_arm/arm_server2.py b/demos/web_arm/arm_server2.py
@@ -1,25 +1,25 @@
-import usb_arm
+import owi_maplin_robot_arm
 from flask import Flask, Response, request, render_template
 
 
-arm = usb_arm.Arm()
+arm = owi_maplin_robot_arm.Arm()
 app = Flask(__name__)
 
 movemap = {
-    "GripsClose": usb_arm.GripsClose,
-    "CloseGrips": usb_arm.CloseGrips,
-    "GripsOpen": usb_arm.GripsOpen,
-    "OpenGrips": usb_arm.OpenGrips,
-    "Stop": usb_arm.Stop,
-    "WristUp": usb_arm.WristUp,
-    "WristDown": usb_arm.WristDown,
-    "ElbowUp": usb_arm.ElbowUp,
-    "ElbowDown": usb_arm.ElbowDown,
-    "ShoulderUp": usb_arm.ShoulderUp,
-    "ShoulderDown": usb_arm.ShoulderDown,
-    "BaseClockWise": usb_arm.BaseClockWise,
-    "BaseCtrClockWise": usb_arm.BaseCtrClockWise,
-    "LedOn": usb_arm.LedOn
+    "GripsClose": owi_maplin_robot_arm.GripsClose,
+    "CloseGrips": owi_maplin_robot_arm.CloseGrips,
+    "GripsOpen": owi_maplin_robot_arm.GripsOpen,
+    "OpenGrips": owi_maplin_robot_arm.OpenGrips,
+    "Stop": owi_maplin_robot_arm.Stop,
+    "WristUp": owi_maplin_robot_arm.WristUp,
+    "WristDown": owi_maplin_robot_arm.WristDown,
+    "ElbowUp": owi_maplin_robot_arm.ElbowUp,
+    "ElbowDown": owi_maplin_robot_arm.ElbowDown,
+    "ShoulderUp": owi_maplin_robot_arm.ShoulderUp,
+    "ShoulderDown": owi_maplin_robot_arm.ShoulderDown,
+    "BaseClockWise": owi_maplin_robot_arm.BaseClockWise,
+    "BaseCtrClockWise": owi_maplin_robot_arm.BaseCtrClockWise,
+    "LedOn": owi_maplin_robot_arm.LedOn
 }
 
 
diff --git a/owi_maplin_usb_arm/__init__.py b/owi_maplin_usb_arm/__init__.py
diff --git a/setup.py b/setup.py
@@ -1,9 +1,9 @@
 from distutils.core import setup
 
 setup(
-    name='usb_robot_arm',
-    version='0.2.0',
-    packages=['usb_robot_arm',],
+    name='owi_maplin_usb_arm',
+    version='0.4.0',
+    packages=['owi_maplin_usb_arm',],
     author="Danny Staple - Orionrobots",
     license='Creative Commons By Attribution Share-Alike v3.0',
     long_description=open('README.md').read(),
diff --git a/setup_arm.sh b/setup_arm.sh
@@ -1,6 +1,6 @@
 #!/bin/env bash -e
 # Raspbian setup
-apt-get update -y 
+apt-get update -y
 apt-get install -y python3-libusb1 python3-pip git
 pip3 install git+https://github.com/orionrobots/python_usb_robot_arm
 cat <<END >/etc/udev/rules.d/42-usb-arm-permissions.rules
diff --git a/tests/test_usb_arm.py b/tests/test_usb_arm.py
