-
Notifications
You must be signed in to change notification settings - Fork 394
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
jupytext gives incorrect warning "You have passed a file name to the '--to' option" #901
Comments
Hi @jli again, Yes you're right, the warning in your case is a bit too much. It is triggered by this condition: Lines 450 to 457 in 57d83aa
I believe I have not seen this in the Jupytext examples because in my examples the suffix start with a dot. I will see how to improve this, meanwhile you can either ignore the warning, or use a suffix that starts with |
Ah, I see, so the example suffixes look like Thanks for the quick response! |
From version 1.13.6 on, the warning will not appear any more when the suffix start with either |
I want to configure jupytext to save companion
.py
files for all.ipynb
notebooks. I want the.py
files to have a specific suffix, so that it's easy to identify those files.This command works:
jupytext --from ipynb --to __sidecar.py:percent <notebook file>
However, I get the following warning from jupytext:
I'm guessing it's this code:
jupytext/jupytext/cli.py
Lines 463 to 466 in 57d83aa
And I'm guessing the issue is that the check
"." in args.output_format
is being triggered?Is something wrong with my --to option? Based on the docs (https://jupytext.readthedocs.io/en/latest/config.html), I believe the format above (
__sidecar.py:percent
) is valid.Is there a workaround I can use in the meantime? It seems that adding
//
could fix this, though I don't quite understand how to use it, or if that will cause side-effects?The text was updated successfully, but these errors were encountered: