Skip to content

Commit

Permalink
ci: run specific notebooks from GitHub Actions (#154)
Browse files Browse the repository at this point in the history
  • Loading branch information
redeboer committed Jun 7, 2022
1 parent 55f3da2 commit 715ce7a
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 8 deletions.
7 changes: 6 additions & 1 deletion .github/workflows/notebooks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,11 @@ name: Run all notebooks

on:
workflow_dispatch:
inputs:
notebook-selector:
description: Relative path to notebooks
required: false
type: string

jobs:
pytest:
Expand All @@ -16,4 +21,4 @@ jobs:
run: |
sudo apt-get -y install graphviz
pip install -c .constraints/py3.8.txt .[test]
- run: pytest --nbmake
- run: pytest --nbmake ${{ github.event.inputs.notebook-selector }}
11 changes: 6 additions & 5 deletions .readthedocs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,11 @@ formats:

build:
os: ubuntu-20.04
tools:
python: "3.8"
apt_packages:
- graphviz
jobs:
post_install:
- pip install -e .[doc] -c .constraints/py3.8.txt
tools:
python: "3.8"
commands:
- pip install -e .[doc] -c .constraints/py3.8.txt
- cat docs/conf.py
- sphinx-build --keep-going -TW -b html docs/ _readthedocs/html
4 changes: 2 additions & 2 deletions docs/adr/001.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,8 @@ requirements:

### Technical story

- {issue}`ComPWA/expertsystem#382`: Coupling parameters in the `AmplitudeModel`
is difficult (has to be done through the place where they are used in the
- {issue}`ComPWA/ampform#5`: Coupling parameters in the `AmplitudeModel` is
difficult (has to be done through the place where they are used in the
`dynamics` or `intensity` section) and counter-intuitive (cannot be done
through the `parameters` section)
- {issue}`ComPWA/expertsystem#440`: when overwriting existing dynamics, old
Expand Down

0 comments on commit 715ce7a

Please sign in to comment.