Skip to content
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

Hook to add alternative notebook managers #2155

Closed
wants to merge 14 commits into from
Closed

Hook to add alternative notebook managers #2155

wants to merge 14 commits into from

Conversation

benjiec
Copy link
Contributor

@benjiec benjiec commented Jul 17, 2012

I added a hook to add alternative notebook managers. See https://github.com/benjiec/django-ipy-nbmgr for an example of a notebook manager that stores notebooks in database, and versions them.

@@ -274,6 +275,9 @@ class NotebookApp(BaseIPythonApplication):
# file to be opened in the notebook server
file_to_run = Unicode('')

# Which alternative NotebookManager to use
nbmgr = Unicode(u'', config=True, help="""Alternative notebook manager module""")
Copy link
Member

Choose a reason for hiding this comment

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

let's follow conventions and call this notebook_manager_class

Copy link
Member

Choose a reason for hiding this comment

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

Note there's also a more specific trait for this sort of thing - DottedObjectName.

@ellisonbg
Copy link
Member

This PR:

#2045

Has a significant refactor of the notebook manager logic to make
things like this easier. Please rebase on top of that branch and
resubmit.

On Tue, Jul 17, 2012 at 8:49 AM, Benjie Chen
reply@reply.github.com
wrote:

I added a hook to add alternative notebook managers. See https://github.com/benjiec/django-ipy-nbmgr for an example of a notebook manager that stores notebooks in database, and versions them.

You can merge this Pull Request by running:

git pull https://github.com/benjiec/ipython master

Or you can view, comment on it, or merge it online at:

#2155

-- Commit Summary --

  • add option to load alternative notebook manager
  • swallow --nbmgr as a frontend arg, and not pass it to kernel

-- File Changes --

M IPython/frontend/html/notebook/notebookapp.py (15)

-- Patch Links --

https://github.com/ipython/ipython/pull/2155.patch
https://github.com/ipython/ipython/pull/2155.diff


Reply to this email directly or view it on GitHub:
#2155

Brian E. Granger
Cal Poly State University, San Luis Obispo
bgranger@calpoly.edu and ellisonbg@gmail.com

bfroehle and others added 12 commits August 3, 2012 14:16
With certain sets of arguments `compile_flags` might be left as `None`. This
caused IPython to internally raise a TypeError when it tried to do a
bitwise or between `shell.compile.flags` and `PyCF_ONLY_AST` in
`CachingCompiler.ast_parse`.

The regression was introduced in:
  b70ac12 embed(): Default to the future compile flags of the calling frame.
track sha of master in test_pr messages
Fix regression in embed() from pull-request #2096.
Fix missing HTML iframe end tags in notebook example.
@Carreau
Copy link
Member

Carreau commented Aug 8, 2012

I see that those commits have changed...
If you tried to rebase you did it on top of master instead of #2045
but as you pushed on master i'll assume that you just push without thinking it would update this PR...

@benjiec
Copy link
Contributor Author

benjiec commented Aug 8, 2012

Yes you are right. It was a "mistake" on my part.

I am investigating if I can just "cancel" the pull request and entirely use
just pull 2045.

On Wed, Aug 8, 2012 at 11:21 AM, Bussonnier Matthias <
notifications@github.com> wrote:

I see that those commits have changed...
If you tried to rebase you did it on top of master instead of #2045#2045
but as you pushed on master i'll assume that you just push without
thinking it would update this PR...


Reply to this email directly or view it on GitHubhttps://github.com//pull/2155#issuecomment-7586890.

@Carreau
Copy link
Member

Carreau commented Aug 8, 2012

Don't worry, you can force-push later or close/reopen with a new branch as target if you wish to.

@benjiec
Copy link
Contributor Author

benjiec commented Aug 8, 2012

I am closing this pull request. Instead pull 2045 works fine with

https://github.com/benjiec/django-ipy-nbmgr

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

Successfully merging this pull request may close these issues.

None yet

6 participants