-
Notifications
You must be signed in to change notification settings - Fork 37
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
Support for Notebook 7 and Jupyter Lab #11
base: main
Are you sure you want to change the base?
Conversation
…, add jupyter lab to requiremnts.txt
…s...eventually replace with Heroku fork) that fixes pgcontents implementation of ContentsManager class to be compatible with Jupyter Lab
… is being deprecated)
…h application to launch, and add JUPYTER_NOTEBOOK_ARGS for additional command line options.
Jupyter lab
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.
Sweet sweet! Added a few commments
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.
Sweet looks great - just need to fix the congif
in the README - can you fix and then land?
README.md
Outdated
@@ -46,6 +48,7 @@ heroku stack:set heroku-24 --app $APP_NAME | |||
|
|||
# Set your required config variable: | |||
heroku config:set JUPYTER_NOTEBOOK_PASSWORD=$JUPYTER_NOTEBOOK_PASSWORD -a $APP_NAME | |||
heroku congif:set JUPYTER_NOTEBOOK_OR_LAB=$JUPYTER_NOTEBOOK_OR_LAB -a $APP_NAME |
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.
congif
- misspelled
I fixed the typo, but I don't have permissions to merge.
…On Fri, Jan 17, 2025 at 8:42 AM Hillary Sanders ***@***.***> wrote:
***@***.**** approved this pull request.
Sweet looks great - just need to fix the congif in the README - can you
fix and then land?
------------------------------
In README.md
<#11 (comment)>:
> @@ -46,6 +48,7 @@ heroku stack:set heroku-24 --app $APP_NAME
# Set your required config variable:
heroku config:set JUPYTER_NOTEBOOK_PASSWORD=$JUPYTER_NOTEBOOK_PASSWORD -a $APP_NAME
+heroku congif:set JUPYTER_NOTEBOOK_OR_LAB=$JUPYTER_NOTEBOOK_OR_LAB -a $APP_NAME
congif - misspelled
—
Reply to this email directly, view it on GitHub
<#11 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AUV3HQXFYMAKPCOZVGYUAZT2LEXI3AVCNFSM6AAAAABUYSE3B2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKNJZGQ3TEOBXGA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Updated repository now launches either Jupyter Notebook 7 or Jupyter Lab on start up (based on env variable JUPYTER_NOTEBOOK_OR_LAB).
This PR includes breaking changes: