Skip to content

Commit

Permalink
Refactor conda installation to use environment file
Browse files Browse the repository at this point in the history
The previous conda installation commands in the 'conda-installs-for-m1.sh' file and '.github/workflows/test.yml' have been removed. Instead, the necessary dependencies are now defined in a new 'environment.yml' file. This simplification will enhance readability and maintainability of the code.
  • Loading branch information
pellet committed Jun 29, 2024
1 parent efe6caf commit 2807d9d
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 6 deletions.
6 changes: 1 addition & 5 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@ jobs:
- name: Install conda
uses: conda-incubator/setup-miniconda@v3
with:
environment-file: environment.yml
auto-activate-base: true
python-version: 3.8
activate-environment: EEG-ExPy
Expand All @@ -65,11 +66,6 @@ jobs:
run: |
conda info
conda activate EEG-ExPy
# install pytables for macOS arm64, so do not need to build from source.
conda install -y pytables
# install liblsl to prevent error on macOS and Ubuntu:
# "RuntimeError: LSL binary library file was not found."
conda install -y liblsl
- name: Install dependencies
shell: bash -el {0}
run: |
Expand Down
1 change: 0 additions & 1 deletion conda-installs-for-m1.sh

This file was deleted.

6 changes: 6 additions & 0 deletions environment.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
channels:
- conda-forge
dependencies:
- python=3.8
- pytables # install pytables for macOS arm64, so do not need to build from source.
- liblsl # install liblsl to prevent error on macOS and Ubuntu: "RuntimeError: LSL binary library file was not found."

0 comments on commit 2807d9d

Please sign in to comment.