Merge pull request #395 from NeuroML/feat/ci-fixes

Deps and CI fixes

Author: sanjayankur31

.github/workflows/ci.yml

@@ -7,7 +7,7 @@ on:
       - development
       - experimental
       - 'test*'
-  pull_request_target:
+  pull_request:
     branches:
       - master
       - development
@@ -18,8 +18,6 @@ jobs:
   build:
 
     runs-on: ubuntu-latest
-    permissions:
-      pull-requests: write
     strategy:
       matrix:
         python-version: ["3.8", "3.9", "3.10", "3.11", "3.12"]
@@ -53,24 +51,3 @@ jobs:
         pynml -h
 
         ./test-ghactions.sh -neuron
-
-    - name: Code Coverage Summary
-      uses: irongut/[email protected]
-      if: ${{ matrix.python-version == '3.12' }}
-      with:
-        output: both
-        filename: coverage.xml
-        badge: true
-        format: markdown
-        indicators: true
-        fail_below_min: false
-        hide_complexity: true
-        hide_branch_rate: true
-        thresholds: '50 75'
-
-    - name: Add Coverage PR Comment
-      uses: marocchino/sticky-pull-request-comment@v2
-      if: ${{ github.event_name == 'pull_request_target' && matrix.python-version == '3.12' }}
-      with:
-        recreate: true
-        path: code-coverage-results.md

setup.cfg

@@ -32,6 +32,7 @@ install_requires =
     graphviz
     typing; python_version<"3.5"
     lxml
+    numpy<2.0.0
     sympy
     ppft[dill]
 
@@ -96,7 +97,7 @@ vispy-common =
     vispy>=0.13.0
     scipy
     pyopengl
-    PyOpenGL-accelerate
+    PyOpenGL-accelerate; python_version<"3.12"
 
 vispy-qt5 =
     pyNeuroML[vispy-common]