-
Notifications
You must be signed in to change notification settings - Fork 72
fix: Set TENSORBOARD_PROXY_URL so TensorBoard can easily be used in JupyterLab #699
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
Conversation
30e64d9 to
a598b3f
Compare
| // These hooks will attempt to persist conda, pip, and jupyterlab extensions between pause and shut-down. | ||
| func Up20201031165106(tx *sql.Tx) error { | ||
| // This code is executed when the migration is applied. | ||
| return updateWorkspaceTemplateManifest("20201031165106_jupyterlab.yaml", jupyterLabTemplateName) |
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.
@aleksandrmelnikov @Vafilor I think for the YAML name, we should have the timestamp first so things are ordered in the IDE accordingly. Thoughts?
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.
@rushtehrani Good point, yeah, let's do that.
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.
Agreed
| // Down20201031165106 removes the lifecycle hooks from the template. | ||
| func Down20201031165106(tx *sql.Tx) error { | ||
| // This code is executed when the migration is rolled back. | ||
| return updateWorkspaceTemplateManifest("20201028145442_jupyterlab.yaml", jupyterLabTemplateName) |
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.
@aleksandrmelnikov @Vafilor When doing down migrations, I'm proposing we use the previous YAML here. Thoughts, issues?
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.
I think that's fine.
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.
Sure
What this PR does:
This allows TensorBoard to be used in Onepanel's JupyterLab Workspaces.
Special notes for your reviewer:
Example usage in JupyterLab:
Checklist
Please check if applies
Required