-
Notifications
You must be signed in to change notification settings - Fork 206
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* Add linting yml-files. * Linted files with black==23.1.0 * Updated the workflows in running the notebooks * Set python versions: ["3.10", "3.11", "3.12"]. * Set continue-on-error to false.
- Loading branch information
Showing
51 changed files
with
1,881 additions
and
1,592 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,46 @@ | ||
name: Linting Notebooks with Black | ||
|
||
on: | ||
push: | ||
branches: | ||
- master # Change this to the repository's main branch | ||
pull_request: | ||
branches: | ||
- master | ||
jobs: | ||
lint: | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: Checkout code | ||
uses: actions/checkout@v4 | ||
|
||
- name: Set up Python | ||
uses: actions/setup-python@v4 | ||
with: | ||
python-version: "3.12" # Change this to your desired Python version | ||
|
||
- name: Install dependencies | ||
run: | | ||
pip install black==23.1.0 nbformat black[jupyter] | ||
continue-on-error: false | ||
|
||
- name: Find notebooks | ||
id: find-notebooks | ||
run: | | ||
find . -name "*.ipynb" > notebooks.txt | ||
continue-on-error: true | ||
|
||
- name: Lint notebooks | ||
run: | | ||
cat notebooks.txt | xargs -I {} black --line-length 88 --check {} | ||
continue-on-error: true | ||
|
||
- name: Check lint results | ||
run: | | ||
if grep -q "would reformat" notebooks.txt; then | ||
echo "Linting issues found. Run 'black' to auto-format the notebooks." | ||
exit 1 | ||
else | ||
echo "All notebooks are properly formatted." | ||
fi |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,65 @@ | ||
name: Run Notebooks | ||
|
||
on: | ||
push: | ||
branches: | ||
- master # Change this to the repository's main branch | ||
pull_request: | ||
branches: | ||
- master | ||
schedule: | ||
- cron: "0 0 * * 0" # Run every Sunday at midnight UTC (4 week interval) | ||
|
||
jobs: | ||
build: | ||
runs-on: ubuntu-latest | ||
strategy: | ||
matrix: | ||
python-version: ["3.10", "3.11", "3.12"] | ||
|
||
steps: | ||
- name: Checkout code | ||
uses: actions/checkout@v4 | ||
|
||
- name: Set up Python ${{ matrix.python-version }} | ||
uses: actions/setup-python@v4 | ||
with: | ||
python-version: ${{ matrix.python-version }} | ||
|
||
# Testing matrix by printing the current Python version: | ||
- name: Display Python version | ||
run: python -c "import sys; print(sys.version)" | ||
|
||
- name: Install dependencies | ||
run: pip install jupyter nbconvert | ||
- run: pip install -r requirements.txt | ||
- run: pip install -r ci/requirements.txt | ||
|
||
- name: Install PortAudio library | ||
run: sudo apt-get install libportaudio2 | ||
- run: sudo apt-get install libasound-dev | ||
|
||
- name: Find notebooks | ||
id: find-notebooks | ||
run: | | ||
find . -name "*.ipynb" -not -name "acoustic_impulse_response_measurement.ipynb" > notebooks.txt | ||
cat notebooks.txt | ||
shell: bash | ||
|
||
- name: Execute notebooks | ||
run: | | ||
cat notebooks.txt | while read -r notebook; do | ||
jupyter nbconvert --to notebook --execute --inplace "$notebook" | ||
done | ||
continue-on-error: false | ||
shell: bash | ||
|
||
- name: Check for errors | ||
run: | | ||
if grep "raise Exception(" *.ipynb; then | ||
echo "Error found in notebook(s)." | ||
exit 1 | ||
else | ||
echo "No errors found in notebooks." | ||
fi | ||
shell: bash |
Oops, something went wrong.