Skip to content
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

Simplify backend dependencies #906

Merged
merged 14 commits into from
Aug 5, 2024
6 changes: 6 additions & 0 deletions .github/workflows/testing_dev.yml
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,12 @@ jobs:
- name: Install GUIDE
run: npm ci --verbose

- name: Install testing dependencies
run: pip install pytest pytest-cov

- name: Manually remove matplotlib
run: pip uninstall matplotlib --yes

- if: matrix.os != 'ubuntu-latest'
name: Run tests
run: npm run test:coverage
Expand Down
6 changes: 6 additions & 0 deletions .github/workflows/testing_dev_with_live_services.yml
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,12 @@ jobs:
- name: Install GUIDE
run: npm ci --verbose

- name: Install testing dependencies
run: pip install pytest pytest-cov

- name: Manually remove matplotlib
run: pip uninstall matplotlib --yes

- name: Create env file
run: |
touch .env
Expand Down
9 changes: 8 additions & 1 deletion .github/workflows/testing_flask_build_and_dist.yml
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,14 @@ jobs:
with:
node-version: "20"

- run: npm ci --verbose
- name: Install GUIDE
run: npm ci --verbose

- name: Install testing dependencies
run: pip install pytest pytest-cov

- name: Manually remove matplotlib
run: pip uninstall matplotlib --yes

# Fix for macos build - remove bad sonpy file
- if: matrix.os == 'macos-latest' || matrix.os == 'macos-13'
Expand Down
6 changes: 6 additions & 0 deletions .github/workflows/testing_pipelines.yml
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,12 @@ jobs:
- name: Install GUIDE
run: npm ci --verbose

- name: Install testing dependencies
run: pip install pytest pytest-cov

- name: Manually remove matplotlib
run: pip uninstall matplotlib --yes

# Load example data caches
- name: Get ephy_testing_data current head hash
id: ephys
Expand Down
11 changes: 4 additions & 7 deletions environments/environment-Linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,7 @@ dependencies:
- flask == 2.3.2
- flask-cors == 4.0.0
- flask_restx == 1.1.0
- neuroconv @ git+https://github.com/catalystneuro/neuroconv.git@main#neuroconv[full]
- dandi >= 0.60.0
- pytest == 7.4.0
- pytest-cov == 4.1.0
- scikit-learn == 1.4.0
- tqdm_publisher >= 0.0.1
- tzlocal >= 5.2
- neuroconv @ git+https://github.com/catalystneuro/neuroconv.git@main#neuroconv[dandi,compressors,ecephys,ophys,behavior,text]
- scikit-learn == 1.4.0 # Tutorial data generation
- tqdm_publisher >= 0.0.1 # Progress bars
- tzlocal >= 5.2 # Frontend timezone handling
27 changes: 12 additions & 15 deletions environments/environment-MAC-apple-silicon.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,13 @@ channels:
dependencies:
- python = 3.9.18
- nodejs = 18.16.1
# Install these from conda-forge so that dependent packages get included in the distributable
- numcodecs = 0.11.0
- lxml = 4.9.3 # pypi build fails due to x64/arm64 mismatch so install from conda-forge
- pyedflib = 0.1.32 # pypi build fails due to x64/arm64 mismatch so install from conda-forge
- numpy # may have x64/arm64 mismatch issues so install from conda-forge
- pytables = 3.8 # pypi build fails on arm64 so install from conda-forge (used by neuroconv deps)
# install these from conda-forge so that dependent packages get included in the distributable
- jsonschema = 4.18.0 # installs jsonschema-specifications
- lxml = 4.9.3 # PyPI build fails due to x64/arm64 mismatch so install from conda-forge
- pyedflib = 0.1.32 # PyPI build fails due to x64/arm64 mismatch so install from conda-forge
- numpy # May have x64/arm64 mismatch issues so install from conda-forge
- pytables = 3.8 # PyPI build fails on arm64 so install from conda-forge (used by neuroconv deps)
- jsonschema = 4.18.0 # Also installs jsonschema-specifications
- pip
- pip:
- setuptools==70.0.0
Expand All @@ -21,12 +21,9 @@ dependencies:
- flask == 2.3.2
- flask-cors == 4.0.0
- flask_restx == 1.1.0
# NOTE: the neuroconv wheel on pypi includes sonpy which is not compatible with arm64, so build and install
# neuroconv from github, which will remove the sonpy dependency when building from mac arm64
- neuroconv @ git+https://github.com/catalystneuro/neuroconv.git@main#neuroconv[full]
- dandi >= 0.60.0
- pytest == 7.4.0
- pytest-cov == 4.1.0
- scikit-learn == 1.4.0
- tqdm_publisher >= 0.0.1
- tzlocal >= 5.2
# NOTE: the NeuroConv wheel on PyPI includes sonpy which is not compatible with arm64, so build and install
# NeuroConv from GitHub, which will remove the sonpy dependency when building from Mac arm64
- neuroconv @ git+https://github.com/catalystneuro/neuroconv.git@main#neuroconv[dandi,compressors,ecephys,ophys,behavior,text]
- scikit-learn == 1.4.0 # Tutorial data generation
- tqdm_publisher >= 0.0.1 # Progress bars
- tzlocal >= 5.2 # Frontend timezone handling
11 changes: 4 additions & 7 deletions environments/environment-MAC-intel.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,7 @@ dependencies:
- flask == 2.3.2
- flask-cors == 4.0.0
- flask_restx == 1.1.0
- neuroconv @ git+https://github.com/catalystneuro/neuroconv.git@main#neuroconv[full]
- dandi >= 0.60.0
- pytest == 7.4.0
- pytest-cov == 4.1.0
- scikit-learn == 1.4.0
- tqdm_publisher >= 0.0.1
- tzlocal >= 5.2
- neuroconv @ git+https://github.com/catalystneuro/neuroconv.git@main#neuroconv[dandi,compressors,ecephys,ophys,behavior,text]
- scikit-learn == 1.4.0 # Tutorial data generation
- tqdm_publisher >= 0.0.1 # Progress bars
- tzlocal >= 5.2 # Frontend timezone handling
11 changes: 4 additions & 7 deletions environments/environment-Windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,7 @@ dependencies:
- flask == 2.3.2
- flask-cors === 3.0.10
- flask_restx == 1.1.0
- neuroconv @ git+https://github.com/catalystneuro/neuroconv.git@main#neuroconv[full]
- dandi >= 0.60.0
- pytest == 7.2.2
- pytest-cov == 4.1.0
- scikit-learn == 1.4.0
- tqdm_publisher >= 0.0.1
- tzlocal >= 5.2
- neuroconv @ git+https://github.com/catalystneuro/neuroconv.git@main#neuroconv[dandi,compressors,ecephys,ophys,behavior,text]
- scikit-learn == 1.4.0 # Tutorial data generation
- tqdm_publisher >= 0.0.1 # Progress bars
- tzlocal >= 5.2 # Frontend timezone handling
Loading