Skip to content

Commit

Permalink
chore: remove need for requirements.txt
Browse files Browse the repository at this point in the history
Now that templateflow will auto-sync and pyntcloud has been released as
a package on pypi, the requirements.txt file is no longer needed to
build the Docker image (which was its unique purpose).
  • Loading branch information
oesteban committed May 27, 2020
1 parent a112d64 commit 601267e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 8 deletions.
10 changes: 4 additions & 6 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -147,20 +147,18 @@ RUN python -c "from matplotlib import font_manager" && \
sed -i 's/\(backend *: \).*$/\1Agg/g' $( python -c "import matplotlib; print(matplotlib.matplotlib_fname())" )

# Installing dev requirements (packages that are not in pypi)
WORKDIR /src/
COPY requirements.txt requirements.txt
RUN pip install --no-cache-dir -r requirements.txt && \
rm -rf $HOME/.cache/pip requirements.txt

# Precaching atlases
RUN python -c "from templateflow import api as tfapi; \
RUN pip install --no-cache-dir "templateflow >= 0.6" && \
rm -rf $HOME/.cache/pip && \
python -c "from templateflow import api as tfapi; \
tfapi.get(['MNI152Lin', 'MNI152NLin2009cAsym', 'OASIS30ANTs'], suffix='T1w'); \
tfapi.get(['MNI152Lin', 'MNI152NLin2009cAsym', 'OASIS30ANTs'], desc='brain', suffix='mask'); \
tfapi.get(['MNI152NLin2009cAsym'], desc='fMRIPrep', suffix='boldref'); \
tfapi.get('OASIS30ANTs', resolution=1, desc='4', suffix='dseg'); \
tfapi.get(['OASIS30ANTs', 'NKI'], resolution=1, label='brain', suffix='probseg'); \
tfapi.get(['OASIS30ANTs', 'NKI'], resolution=1, desc='BrainCerebellumRegistration', suffix='mask'); "

WORKDIR /src/
COPY . niworkflows/
WORKDIR /src/niworkflows/
RUN pip install --no-cache-dir -e .[all] && \
Expand Down
2 changes: 0 additions & 2 deletions requirements.txt

This file was deleted.

0 comments on commit 601267e

Please sign in to comment.