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

pass extra arguments to ipython kernel #70

Merged
merged 1 commit into from Jun 28, 2017

Conversation

dvzubarev
Copy link
Contributor

It was removed in commit 5b8bb63
Now arguments are passed to the driver and then to the kernel.
I use it to specify a profile for kernels, since I have a profile for org-mode that converts dict, tuples, pd.Series to org-mode tables.

@gregsexton
Copy link
Owner

Can you rebase and test this again? I think it may clash with the change I just merged.

@gregsexton gregsexton self-assigned this Jul 3, 2016
@dvzubarev
Copy link
Contributor Author

I've rebased and tested, works fine for me.
I've been using ob-ipython with merged #68 for a while.
There were no problems for me.

@gregsexton
Copy link
Owner

Ok, thanks. I've assigned this to myself which means I will take a look at it when I get chance.

@gregsexton gregsexton merged commit 167fef2 into gregsexton:master Jun 28, 2017
@gregsexton
Copy link
Owner

Sorry this took so long. Really appreciate the PR.

@dcherian
Copy link

I use it to specify a profile for kernels, since I have a profile for org-mode that converts dict, tuples, pd.Series to org-mode tables.

@dvzubarev can you post this profile of yours please?

@dvzubarev dvzubarev deleted the fix_missing_extra_args branch June 29, 2017 09:30
@dvzubarev
Copy link
Contributor Author

@dcherian I created new profile based on tabulated. See for more code and examples.

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

3 participants