-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updates to Interactive Environment docs #2543
Conversation
* Move docs on writing IEs to developer docs
docker containers. Once this container is launched, we notify a proxy built | ||
into Galaxy which helps coordinate a 1:1 mapping of users and their docker containers. | ||
component. Galaxy vizualisation plugins are rendered using Mako templates and | ||
Mako templates in turn can run Python code. This Python code in the Mako templates |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"code. This" -> "code. GIEs build upon visualization plugins, adding features to allow for container management and proxying. This"
Also you'll need to mention this in docs/dev/index.rst or something, I think |
location /galaxy/gie_proxy/ipython/api/kernels { | ||
proxy_pass http://localhost:8800/galaxy/gie_proxy/ipython/api/kernels; | ||
location ~ ^/galaxy/gie_proxy/jupyter/(?<nbtype>[^/]+)/api/kernels(?<rest>.*?)$ { | ||
proxy_pass http://localhost:8800/galaxy/gie_proxy/$nbtype/api/kernels$rest; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bgruening, @pvanheus says this is correct and I believe him, but just wanted a second opinion :)
After @bgruening replies, ready to merge |
@@ -3,21 +3,23 @@ Galaxy Interactive Environments (GIEs) | |||
|
|||
GIEs were a new feature back in Galaxy 15.05, leading with the release of the | |||
IPython IE. They were presented at GCC2015, and the RStudio IE was released as | |||
part of 15.07. | |||
part of 15.07. The IPython IE was superceded by a Project Jupyter IE in | |||
Galaxy XXXXX. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
16.01
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ping @pvanheus
Thanks for enhancing it!
Awesome - thanks for the continued improvements to docs @pvanheus! |
Add updates to the Interactive Environment docs and split out the docs on writing IEs to developer documentation.