jgomezdans / geog_docker

Docker image for Geospatial Python teaching
GNU General Public License v3.0
4 stars 1 forks source link

Moans ;) #29

Closed jgomezdans closed 4 years ago

jgomezdans commented 4 years ago

Can you pls add a

conda update -n uclgeog --all --yes

to update at the end to save that from taking too long in my docker ;-)

and add git to the apt packages ...

and do apt-update ...

Thanks

Lewis

jgomezdans commented 4 years ago
  1. git is already there (derive your image from jgomezdans/conda_base, see the geog0666 dockerfile, the apt cache is also updatedd there.
  2. Not sure what you mean by update uclgeog... The conda install is already updated in conda_base

You only need to change the second target in the dockerfile, not the first one. The idea is to re-use the first one for all images and to save disk/bandwidth like that.

The base image only has jupyter installed. I tried adding numpy & matplotlib but this forces all sorts of versions as different things might have the numpy version hardwired. This is an issue as the number of packages in the environment.yml file goes up (increases the chance of version collision)

profLewis commented 4 years ago

Hi

So all I have now is


FROM jgomezdans/conda_base

LABEL maintainer="Philip Lewis p.lewis@ucl.ac.uk" ENV GEOG0111_VERSION 0.0.1

USER $NB_USER

RUN git clone https://github.com/profLewis/geog0111-core.git RUN mkdir -p $HOME/geog0111-core/notebooks/oneDrive WORKDIR $HOME/geog0111-core/notebooks

update conda packages with my environment.yml

RUN conda update --all —yes

switch off ugly stuff

RUN jupyter nbextension disable execute_time/ExecuteTime

OK?

On 5 Aug 2020, at 15:31, José Gómez-Dans notifications@github.com wrote:

git is already there (derive your image from jgomezdans/conda_base, see the geog0666 dockerfile, the apt cache is also updatedd there. Not sure what you mean by update uclgeog... The conda install is already updated in conda_base You only need to change the second target in the dockerfile, not the first one. The idea is to re-use the first one for all images and to save disk/bandwidth like that.

The base image only has jupyter installed. I tried adding numpy & matplotlib but this forces all sorts of versions as different things might have the numpy version hardwired. This is an issue as the number of packages in the environment.yml file goes up (increases the chance of version collision)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jgomezdans/geog_docker/issues/29#issuecomment-669228124, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOC3GXGLFZ7BBN5SKCGAKTR7FUN5ANCNFSM4PVRH72A.

profLewis commented 4 years ago

conda-base ... I take it

Professor Philip Lewis, UCL

On 5 Aug 2020, at 15:38, Philip Lewis dr.p.e.lewis@gmail.com wrote:

Hi

So all I have now is


FROM jgomezdans/conda_base

LABEL maintainer="Philip Lewis p.lewis@ucl.ac.uk" ENV GEOG0111_VERSION 0.0.1

USER $NB_USER

RUN git clone https://github.com/profLewis/geog0111-core.git RUN mkdir -p $HOME/geog0111-core/notebooks/oneDrive WORKDIR $HOME/geog0111-core/notebooks

update conda packages with my environment.yml

RUN conda update --all —yes

switch off ugly stuff

RUN jupyter nbextension disable execute_time/ExecuteTime

OK?

On 5 Aug 2020, at 15:31, José Gómez-Dans notifications@github.com wrote:

git is already there (derive your image from jgomezdans/conda_base, see the geog0666 dockerfile, the apt cache is also updatedd there. Not sure what you mean by update uclgeog... The conda install is already updated in conda_base You only need to change the second target in the dockerfile, not the first one. The idea is to re-use the first one for all images and to save disk/bandwidth like that.

The base image only has jupyter installed. I tried adding numpy & matplotlib but this forces all sorts of versions as different things might have the numpy version hardwired. This is an issue as the number of packages in the environment.yml file goes up (increases the chance of version collision)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

jgomezdans commented 4 years ago

I'm testing and will do a PR on your repo in a minute, ensuring everything works first

On Wed, 5 Aug 2020 at 15:47, Professor Philip Lewis < notifications@github.com> wrote:

conda-base ... I take it

Professor Philip Lewis, UCL

On 5 Aug 2020, at 15:38, Philip Lewis dr.p.e.lewis@gmail.com wrote:

Hi

So all I have now is


FROM jgomezdans/conda_base

LABEL maintainer="Philip Lewis p.lewis@ucl.ac.uk" ENV GEOG0111_VERSION 0.0.1

USER $NB_USER

RUN git clone https://github.com/profLewis/geog0111-core.git RUN mkdir -p $HOME/geog0111-core/notebooks/oneDrive WORKDIR $HOME/geog0111-core/notebooks

update conda packages with my environment.yml

RUN conda update --all —yes

switch off ugly stuff

RUN jupyter nbextension disable execute_time/ExecuteTime

OK?

On 5 Aug 2020, at 15:31, José Gómez-Dans notifications@github.com wrote:

git is already there (derive your image from jgomezdans/conda_base, see the geog0666 dockerfile, the apt cache is also updatedd there. Not sure what you mean by update uclgeog... The conda install is already updated in conda_base You only need to change the second target in the dockerfile, not the first one. The idea is to re-use the first one for all images and to save disk/bandwidth like that.

The base image only has jupyter installed. I tried adding numpy & matplotlib but this forces all sorts of versions as different things might have the numpy version hardwired. This is an issue as the number of packages in the environment.yml file goes up (increases the chance of version collision)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/jgomezdans/geog_docker/issues/29#issuecomment-669236746, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABCAKEPJIC6XLUOTOH2ZALR7FWINANCNFSM4PVRH72A .

-- RSU ■ Dept. of Geography ■ University College ■ Gower St, London WC1E 6BT UK EMM ■ Dept. of Geography ■ King's College ■ Strand, London WC2R 2LS UK