Skip to content

Commit 1b7ed06

Browse files
committed
Fix version selection
Previously, the simulator version argument was ignored and always the latest amici version was installed.
1 parent abd1120 commit 1b7ed06

File tree

4 files changed

+29
-41
lines changed

4 files changed

+29
-41
lines changed

.github/workflows/ci.yml

Lines changed: 14 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -75,18 +75,18 @@ jobs:
7575
with:
7676
python-version: '3.11'
7777

78-
- name: Setup pip cache
79-
uses: actions/cache@v3
80-
with:
81-
path: /opt/hostedtoolcache/Python
82-
key: ${{ runner.os }}-pip-${{ hashFiles('requirements.txt') }}-${{ hashFiles('requirements.optional.txt') }}
83-
restore-keys: |
84-
${{ runner.os }}-pip-
78+
# - name: Setup pip cache
79+
# uses: actions/cache@v3
80+
# with:
81+
# path: /opt/hostedtoolcache/Python
82+
# key: ${{ runner.os }}-pip-${{ hashFiles('requirements.txt') }}-${{ hashFiles('requirements.optional.txt') }}
83+
# restore-keys: |
84+
# ${{ runner.os }}-pip-
8585

8686
- name: Install pip and setuptools
8787
run: |
88-
python -m pip install --upgrade pip==23.0.0
89-
python -m pip install --upgrade pip==23.0.0
88+
python -m pip install --upgrade setuptools
89+
python -m pip install --upgrade pip
9090
9191
- name: Install system dependencies
9292
run: |
@@ -98,7 +98,11 @@ jobs:
9898
9999
# install package
100100
- name: Install the package
101-
run: python -m pip install .[all]
101+
run: |
102+
simulatorVersion=$(jq -r '.version' biosimulators.json)
103+
python -m pip install amici==$simulatorVersion
104+
python -m pip install pkg_utils # for current setup.py
105+
python -m pip install .[all]
102106
103107
#############################################
104108
## Lint

Dockerfile

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,11 +46,12 @@ RUN apt-get update -y \
4646

4747
# Copy code for command-line interface into image and install it
4848
COPY . /root/Biosimulators_AMICI
49-
RUN pip install pip==23.0.1
50-
RUN pip install sympy /root/Biosimulators_AMICI \
49+
RUN python -m pip install -U setuptools pip
50+
RUN python -m pip install pkg_utils # for current setup.py
51+
52+
RUN python -m pip install /root/Biosimulators_AMICI \
5153
&& rm -rf /root/Biosimulators_AMICI
52-
#RUN pip install sympy /root/Biosimulators_AMICI amici==${SIMULATOR_VERSION} \
53-
# && rm -rf /root/Biosimulators_AMICI
54+
RUN python -m pip install amici==${SIMULATOR_VERSION}
5455
ENV VERBOSE=0 \
5556
MPLBACKEND=PDF
5657

biosimulators.json

Lines changed: 10 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"id": "amici",
33
"name": "AMICI",
4-
"version": "0.24.0",
4+
"version": "0.34.2",
55
"description": "AMICI provides an interface for the SUNDIALS solvers CVODES (for ordinary differential equations) and IDAS (for algebraic differential equations).",
66
"urls": [
77
{
@@ -10,7 +10,7 @@
1010
}
1111
],
1212
"image": {
13-
"url": "ghcr.io/biosimulators/biosimulators_amici/amici:0.24.0",
13+
"url": "ghcr.io/biosimulators/biosimulators_amici/amici:0.34.2",
1414
"format": {
1515
"namespace": "EDAM",
1616
"id": "format_3973",
@@ -100,34 +100,18 @@
100100
],
101101
"citations": [
102102
{
103-
"title": "Scalable parameter estimation for genome-scale biochemical reaction networks",
104-
"authors": "Fabian Fröhlich, Barbara Kaltenbacher, Fabian J. Theis & Jan Hasenauer",
105-
"journal": "PLOS Computational Biology",
106-
"volume": "13",
107-
"issue": "1",
108-
"pages": "e1005331",
109-
"year": 2017,
110-
"identifiers": [
111-
{
112-
"namespace": "doi",
113-
"id": "10.1371/journal.pcbi.1005331",
114-
"url": "https://doi.org/10.1371/journal.pcbi.1005331"
115-
}
116-
]
117-
},
118-
{
119-
"title": "Parameter estimation for dynamical systems with discrete events and logical operations",
120-
"authors": "Fabian Fröhlich, Fabian J. Theis, Joachim O Rädler & Jan Hasenauer",
103+
"title": "AMICI: High-Performance Sensitivity Analysis for Large Ordinary Differential Equation Models",
104+
"authors": "Fabian Fröhlich, Daniel Weindl, Yannik Schälte, Dilan Pathirana, Paszkowski Lukasz, Glenn Terje Lines, Paul Stapor & Jan Hasenauer",
121105
"journal": "Bioinformatics",
122-
"volume": "33",
123-
"issue": "7",
124-
"pages": "1049-1056",
125-
"year": 2017,
106+
"volume": "37",
107+
"issue": "20",
108+
"pages": "3676–3677",
109+
"year": 2021,
126110
"identifiers": [
127111
{
128112
"namespace": "doi",
129-
"id": "10.1093/bioinformatics/btw764",
130-
"url": "https://doi.org/10.1093/bioinformatics/btw764"
113+
"id": "10.1093/bioinformatics/btab227",
114+
"url": "https://doi.org/10.1093/bioinformatics/btab227"
131115
}
132116
]
133117
}

requirements.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
amici >= 0.11.17
21
biosimulators_utils[logging] >= 0.1.180
32
kisao >= 2.33
43
lxml

0 commit comments

Comments
 (0)