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

Command line conversion using --from and --to #46

Closed
mwouts opened this issue Aug 30, 2018 · 2 comments · Fixed by #58 or #60
Closed

Command line conversion using --from and --to #46

mwouts opened this issue Aug 30, 2018 · 2 comments · Fixed by #58 or #60

Comments

@mwouts
Copy link
Owner

mwouts commented Aug 30, 2018

Command line usage should be more similar to jupyter nbconvert.
An additional argument --test-round-trip for testing round trip conversion would be useful as well.

mwouts added a commit that referenced this issue Aug 31, 2018
mwouts added a commit that referenced this issue Aug 31, 2018
@mwouts mwouts mentioned this issue Aug 31, 2018
@mwouts mwouts closed this as completed Aug 31, 2018
@mwouts
Copy link
Owner Author

mwouts commented Sep 5, 2018

Eric recommends to go further in the direction of a standard command line.

jupytext --to python [--output script.py] [--test] notebook.ipynb
jupytext --to notebook [--output notebook.ipynb] [--test] [--update] script.py

See also here for an explicitly required --to option.

The optional -o / --output option could accept a - for stdout output.

@mwouts mwouts reopened this Sep 5, 2018
mwouts added a commit that referenced this issue Sep 5, 2018
mwouts added a commit that referenced this issue Sep 5, 2018
@mwouts mwouts mentioned this issue Sep 5, 2018
@mwouts mwouts closed this as completed in #58 Sep 5, 2018
@mwouts
Copy link
Owner Author

mwouts commented Sep 6, 2018

Command line jupytext only works on Python 3 in pypi's jupytext v0.6.2. The fix should be integrated in the next release.

@mwouts mwouts reopened this Sep 6, 2018
@mwouts mwouts mentioned this issue Sep 7, 2018
@mwouts mwouts closed this as completed in #60 Sep 7, 2018
mwouts added a commit that referenced this issue Sep 7, 2018
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 a pull request may close this issue.

1 participant