-
Notifications
You must be signed in to change notification settings - Fork 48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[INFRA] MATLAB test automation #43
Open
Remi-Gau
wants to merge
30
commits into
cvnlab:main
Choose a base branch
from
Remi-Gau:matlab_test_automation
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+792
−19
Open
Changes from 1 commit
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
af934b4
Add common tools for MATLAB and Python testing
Remi-Gau 20e6fd9
add smoke tests matlab
Remi-Gau 918091b
add matlab test automation
Remi-Gau aee01dc
add matlab test demo automation
Remi-Gau faba452
add expected data and an assert to system test
Remi-Gau b886a8b
rename test file
Remi-Gau 04508a5
add documentation
Remi-Gau b9df9f2
lint
Remi-Gau 0eff103
remove unneeded upload coverage artefact from CI workflow
Remi-Gau 3ab086b
run tests matlab demos with CRON job on 1rt and 15th of every month
Remi-Gau ef3920e
add assert on value of R2
Remi-Gau 8284581
add doc in CI yml and scripts
Remi-Gau 61cfe71
fix typo
Remi-Gau bd5d8fa
add matlab tests badge to README
Remi-Gau 956da88
add matlab demo badge to README
Remi-Gau 9def9a2
run demos with MOxUnit in CI so that demo2 is run even if demo1 fails
Remi-Gau c4530c3
force demos to fail to make sure this is picked by CI
Remi-Gau 10161f0
add a check log to try to show workflow as failed
Remi-Gau f569dd6
remove forced errors in demo and reset to run demos bimonthly
Remi-Gau 26389af
add doc on running tests and demos in CI
Remi-Gau 56807a6
miss_hit lint CI code
Remi-Gau 07da5c3
add miss hit and linting documentation
Remi-Gau 1908552
add pre-commit doc
Remi-Gau 70d205e
update contributors doc
Remi-Gau 731ac95
add doc on generating and chekcing test data
Remi-Gau a312579
update repo map
Remi-Gau b35bec6
switch to using png image in doc
Remi-Gau ca40488
try testing different os and matlab version
Remi-Gau 882cf8e
only run tests on ubuntu latest with matlab 2020a
Remi-Gau 4ae81fd
Update .github/workflows/run_tests_matlab.yaml
Remi-Gau File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next
Next commit
Add common tools for MATLAB and Python testing
Common base for the matlab and python testing branch to minimize merge conflicts down the line
commit af934b43618ad446c251811cc08b5221d26e7aff
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
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,7 @@ | ||
# See https://pre-commit.com for more information | ||
# See https://pre-commit.com/hooks.html for more hooks | ||
repos: | ||
- repo: https://github.com/pre-commit/pre-commit-hooks | ||
rev: v4.1.0 | ||
hooks: | ||
- id: check-yaml |
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,56 @@ | ||
# CONTRIBUTING | ||
|
||
Information for anyone who would like to contribute to this repository. | ||
|
||
## Repository map | ||
|
||
```bash | ||
├── .git | ||
├── .github | ||
│ └── workflows # Github continuous integration set up | ||
├── examples | ||
│ ├── data | ||
│ ├── example1outputs | ||
│ ├── example2outputs | ||
├── glmsingle # Python implementation | ||
│ ├── cod | ||
│ ├── design | ||
│ ├── gmm | ||
│ ├── hrf | ||
│ ├── ols | ||
│ ├── ssq | ||
│ └── utils | ||
├── matlab # Matlab implementation | ||
│ ├── examples | ||
│ ├── fracridge | ||
│ └── utilities | ||
└── tests # Python and Matlab tests | ||
└── data | ||
|
||
``` | ||
|
||
## Generic | ||
|
||
### Makefile | ||
|
||
### pre-commit | ||
|
||
## Matlab | ||
|
||
### Style guide | ||
|
||
### Tests | ||
|
||
#### Demos | ||
|
||
### Continuous integration | ||
|
||
## Python | ||
|
||
### Style guide | ||
|
||
### Tests | ||
|
||
#### Demos | ||
|
||
### Continuous integration |
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,116 @@ | ||
.DEFAULT_GOAL := help | ||
|
||
BROWSER := python -c "$$BROWSER_PYSCRIPT" | ||
|
||
# TODO make more general to use the local matlab version | ||
MATLAB = /usr/local/MATLAB/R2017a/bin/matlab | ||
MATLAB_ARG = -nodisplay -nosplash -nodesktop | ||
|
||
define BROWSER_PYSCRIPT | ||
import os, webbrowser, sys | ||
|
||
from urllib.request import pathname2url | ||
|
||
webbrowser.open("file://" + pathname2url(os.path.abspath(sys.argv[1]))) | ||
endef | ||
export BROWSER_PYSCRIPT | ||
|
||
# determines what "make help" will show | ||
define PRINT_HELP_PYSCRIPT | ||
import re, sys | ||
|
||
for line in sys.stdin: | ||
match = re.match(r'^([a-zA-Z_-]+):.*?## (.*)$$', line) | ||
if match: | ||
target, help = match.groups() | ||
print("%-20s %s" % (target, help)) | ||
endef | ||
export PRINT_HELP_PYSCRIPT | ||
|
||
################################################################################ | ||
# GENERIC | ||
.PHONY: help clean clean-test lint install_dev | ||
|
||
help: | ||
@python -c "$$PRINT_HELP_PYSCRIPT" < $(MAKEFILE_LIST) | ||
clean: clean-build clean-pyc clean-test ## remove all build, test, coverage artifacts | ||
|
||
clean-test: ## remove test and coverage artifacts | ||
rm -rf .tox/ | ||
rm -rf .coverage | ||
rm -rf htmlcov/ | ||
rm -rf .pytest_cache | ||
rm -rf tests/data | ||
rm -rf test/outputs | ||
|
||
install_dev: ## install for both matlab and python developpers | ||
pip install -e . | ||
pip install -r requirements_dev.txt | ||
|
||
lint: lint/black lint/flake8 lint/miss_hit ## check style | ||
|
||
test: test-matlab test-python | ||
|
||
tests/data/nsdcoreexampledataset.mat: | ||
mkdir tests/data | ||
curl -fsSL --retry 5 -o "tests/data/nsdcoreexampledataset.mat" https://osf.io/k89b2/download | ||
|
||
################################################################################ | ||
|
||
################################################################################ | ||
# MATLAB | ||
|
||
.PHONY: lint/miss_hit | ||
|
||
lint/miss_hit: ## lint and checks matlab code | ||
mh_style --fix tests && mh_metric --ci tests && mh_lint tests | ||
|
||
test-matlab: tests/data/nsdcoreexampledataset.mat | ||
$(MATLAB) $(MATLAB_ARG) -r "run_tests; exit()" | ||
|
||
coverage-matlab: test-matlab | ||
$(BROWSER) coverage_html/index.html | ||
|
||
################################################################################ | ||
|
||
################################################################################ | ||
# PYTHON | ||
|
||
.PHONY: clean-build clean-pyc coverage-python install lint/flake8 lint/black | ||
|
||
clean-build: ## remove build artifacts | ||
rm -fr build/ | ||
rm -fr dist/ | ||
rm -fr .eggs/ | ||
find . -name '*.egg-info' -exec rm -fr {} + | ||
find . -name '*.egg' -exec rm -f {} + | ||
|
||
clean-pyc: ## remove Python file artifacts | ||
find . -name '*.pyc' -exec rm -f {} + | ||
find . -name '*.pyo' -exec rm -f {} + | ||
find . -name '*~' -exec rm -f {} + | ||
find . -name '__pycache__' -exec rm -fr {} + | ||
|
||
lint/flake8: ## check style with flake8 | ||
flake8 tests | ||
lint/black: ## check style with black | ||
black tests | ||
|
||
test-python: tests/data/nsdcoreexampledataset.mat ## run tests quickly with the default Python | ||
pytest | ||
|
||
test-notebooks: | ||
pytest --nbmake --nbmake-timeout=3000 "./examples" | ||
test-all: ## run tests on every Python version with tox | ||
tox | ||
|
||
coverage-python: ## check code coverage quickly with the default Python | ||
coverage run --source glmsingle -m pytest | ||
coverage report -m | ||
coverage html | ||
$(BROWSER) htmlcov/index.html | ||
|
||
install: clean ## install the package to the active Python's site-packages | ||
python setup.py install | ||
|
||
################################################################################ |
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
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,11 @@ | ||
# Matlab dev | ||
miss_hit | ||
|
||
# Python dev | ||
flake8 | ||
tox | ||
coverage | ||
pytest | ||
black | ||
nbmake | ||
pytest-cov |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI Reordered the README to have all the install instructions first for MATLAB and Python and then the examples. Can change it back.