Skip to content

Commit

Permalink
start of jupyter notebook server on ip adress '0.0.0.0' instead of de…
Browse files Browse the repository at this point in the history
…fault '*' which is no longer supported

this is probably needed only on a temporary basis
see also
jupyter/docker-stacks#718
https://github.com/ageron/handson-ml/pull/317/commits
  • Loading branch information
parmentelat committed Nov 29, 2018
1 parent 35e22f2 commit 40618c7
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion scripts/nbh
Original file line number Diff line number Diff line change
Expand Up @@ -761,7 +761,7 @@ function docker-create-container-for-student-in-course() {
STUDENT_uid=$(id -u $student)

command="docker create --name $container
-p 8888
--publish 8888
--user root
--env NBAUTOEVAL_LOG=/home/jovyan/work/.nbautoeval
-v $STUDENT_course:/home/jovyan/work
Expand All @@ -774,11 +774,15 @@ function docker-create-container-for-student-in-course() {
done
# see start-in-dir-as-uid.sh for a note on setting the directories
# below
# note that setting the ip here is a workaround
# for a broken jupyter/docker-stacks version around late 2018
# probably not needed with more recent ones
command="$command
-e PYTHONPATH=/home/jovyan/modules
$COURSE_image
start-in-dir-as-uid.sh /home/jovyan $STUDENT_uid
jupyter notebook
--ip=0.0.0.0
--no-browser
--NotebookApp.notebook_dir=/home/jovyan/work
--NotebookApp.token=$jupyter_token
Expand Down

0 comments on commit 40618c7

Please sign in to comment.