Skip to content
This repository has been archived by the owner on Dec 18, 2023. It is now read-only.

tmtk notebook server, SSL proxy for many hosts and Let's Encrypt service #40

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

gijskant
Copy link
Contributor

This branch has been used to setup the environments for the Tübingen workshop. These changes could be useful in the future. Please review if these changes would have negative consequences for other installations, and if these changes can be improved.

README.md Outdated
## Running a Jupyter notebook server

The `startall` script has a flag `--notebook-server` that allows you to run your setup together with a
[Jupyter](https://jupyter.org) note book server with the [tmtk]((https://pypi.org/project/tmtk/)) and [transmart](https://pypi.org/project/transmart/)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note book -> notebook
Extra parenthesis in the tmtk url break the link.

#!/bin/sh

#
# Script to generate configuration for the Let's encrypt service.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe useful to shortly mention this script in the README.md


ARG TMTK_VERSION="0.5.6"
ARG TRANSMART_API_CLIENT_PY_VERSION="0.2.6"
ARG TRANSMART_VERSION="17.2.4"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not make these configurable?

@@ -0,0 +1,20 @@
# Jupyter notebook server with tmtk and TranSMART API client

Image to launch a Jupyter notebook server with the `tmtk` and `transmart`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

transmart packages meaning transmart-api-client package, right?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants