Skip to content

Commit

Permalink
cherry-pick Fix RXD test (#3324) Plotly 6 requires anywidget
Browse files Browse the repository at this point in the history
  • Loading branch information
JCGoran authored and nrnhines committed Jan 31, 2025
1 parent b165176 commit f7a9dc2
Show file tree
Hide file tree
Showing 5 changed files with 16 additions and 2 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ on:
# - 'docs/**'

env:
PY_MIN_VERSION: '3.7'
PY_MIN_VERSION: '3.9'
PY_MAX_VERSION: '3.13'

jobs:
Expand Down Expand Up @@ -63,11 +63,13 @@ jobs:
- name: Install Python@${{ env.PY_MIN_VERSION }} dependencies
working-directory: ${{runner.workspace}}/nrn
run: |
python${PY_MIN_VERSION} -m pip install --upgrade -r ci_requirements.txt
python${PY_MIN_VERSION} -m pip install --upgrade pip -r nrn_requirements.txt
- name: Install Python@${{ env.PY_MAX_VERSION }} dependencies
working-directory: ${{runner.workspace}}/nrn
run: |
python${PY_MAX_VERSION} -m pip install --upgrade -r ci_requirements.txt
python${PY_MAX_VERSION} -m pip install --upgrade pip -r nrn_requirements.txt
- name: Build & Test
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/neuron-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -132,6 +132,7 @@ jobs:
if: ${{ matrix.config.python_dynamic == 'ON' }}
working-directory: ${{runner.workspace}}/nrn
run: |
python -m pip install --upgrade -r ci_requirements.txt
python -m pip install --upgrade pip -r nrn_requirements.txt
- name: Set up Python@${{ env.PY_MAX_VERSION }}
Expand All @@ -142,6 +143,7 @@ jobs:
- name: Install Python@${{ env.PY_MAX_VERSION }} dependencies
working-directory: ${{runner.workspace}}/nrn
run: |
python -m pip install --upgrade -r ci_requirements.txt
python -m pip install --upgrade pip -r nrn_requirements.txt
- name: Setup MUSIC
Expand Down
4 changes: 4 additions & 0 deletions ci_requirements.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
plotly
ipywidgets>=7.0.0
tenacity<8.4 # needed until plotly fixes it upstream
anywidget
1 change: 1 addition & 0 deletions docs/docs_requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -16,3 +16,4 @@ jinja2
sphinx-design
packaging==21.3
tenacity<8.4
anywidget
7 changes: 6 additions & 1 deletion packaging/python/test_requirements.txt
Original file line number Diff line number Diff line change
@@ -1,2 +1,7 @@
pytest
setuptools;python_version>='3.12' # From 3.12, no longer installed by default
pytest<=8.1.1 # potential bug from 8.2.0 due to parallelism?
# for coverage
pytest-cov
# for RXD test
plotly
anywidget

0 comments on commit f7a9dc2

Please sign in to comment.