Skip to content

Commit 55e85ba

Browse files
committed
Remove caching, fix analysis
1 parent 56aa4c1 commit 55e85ba

File tree

1 file changed

+17
-18
lines changed

1 file changed

+17
-18
lines changed

.github/workflows/analysis.yml

Lines changed: 17 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -19,19 +19,17 @@ jobs:
1919
with:
2020
python-version: "3.11"
2121
cache: 'pip'
22-
- name: Cache virtualenv
23-
uses: actions/cache@v3
24-
with:
25-
key: venv-${{ runner.os }}-${{ steps.setup_python.outputs.python-version}}-${{ hashFiles('requirements.txt') }}
26-
path: .venv
22+
# - name: Cache virtualenv
23+
# uses: actions/cache@v3
24+
# with:
25+
# key: venv-${{ runner.os }}-${{ steps.setup_python.outputs.python-version}}-${{ hashFiles('requirements.txt') }}
26+
# path: .venv
2727
- name: Install dependencies
2828
run: |
2929
python -m venv .venv
3030
source .venv/bin/activate
3131
python -m pip install -r requirements.txt
3232
python -m pip install pytest
33-
echo "$VIRTUAL_ENV/bin" >> $GITHUB_PATH
34-
echo "VIRTUAL_ENV=$VIRTUAL_ENV" >> $GITHUB_ENV
3533
- name: Read algorithms
3634
id: algorithms
3735
run: |
@@ -61,11 +59,11 @@ jobs:
6159
with:
6260
python-version: "3.11"
6361
cache: 'pip'
64-
- name: Cache virtualenv
65-
uses: actions/cache@v3
66-
with:
67-
key: venv-${{ runner.os }}-${{ steps.setup_python.outputs.python-version}}-${{ hashFiles('requirements.txt') }}
68-
path: .venv
62+
# - name: Cache virtualenv
63+
# uses: actions/cache@v3
64+
# with:
65+
# key: venv-${{ runner.os }}-${{ steps.setup_python.outputs.python-version}}-${{ hashFiles('requirements.txt') }}
66+
# path: .venv
6967
- name: Install dependencies
7068
run: |
7169
python -m venv .venv
@@ -79,7 +77,8 @@ jobs:
7977
- name: Upload raw data
8078
uses: actions/upload-artifact@v3
8179
with:
82-
name: WorkingData
80+
name: Working_Data
81+
retention-days: 1
8382
path: |
8483
test_output_${{ matrix.algorithm }}_${{ matrix.SNR }}.csv
8584
test_duration_${{ matrix.algorithm }}_${{ matrix.SNR }}.csv
@@ -94,14 +93,14 @@ jobs:
9493
path: artifacts
9594
- name: Merge fitting results
9695
run: |
97-
cat artifacts/WorkingData/test_output_*.csv > test_output.csv
96+
cat artifacts/Working_Data/test_output_*.csv > test_output.csv
9897
- name: Merge timing results
9998
run: |
100-
cat artifacts/WorkingData/test_duration_*.csv > test_duration.csv
99+
cat artifacts/Working_Data/test_duration_*.csv > test_duration.csv
101100
- name: Upload merged artifacts
102101
uses: actions/upload-artifact@v3
103102
with:
104-
name: Combined Data
103+
name: Data
105104
path: |
106105
test_output.csv
107106
test_duration.csv
@@ -127,13 +126,13 @@ jobs:
127126
- name: Download artifacts
128127
uses: actions/download-artifact@v3
129128
with:
130-
name: Raw Data
129+
name: Data
131130
- name: Generate figures
132131
run: Rscript --vanilla tests/IVIMmodels/unit_tests/analyze.r test_output.csv test_duration.csv
133132
- name: Upload figures
134133
uses: actions/upload-artifact@v3
135134
with:
136-
name: Fit Figures
135+
name: Figures
137136
path: |
138137
D.pdf
139138
f.pdf

0 commit comments

Comments
 (0)