vimpyter icon indicating copy to clipboard operation
vimpyter copied to clipboard

fails to load notebooks with spaces in name

Open jwkvam opened this issue 8 years ago • 7 comments

This looks really handy! I often have spaces in my notebook filenames and it seems like that breaks something. It just loads up an empty buffer when I try.

jwkvam avatar Feb 28 '18 19:02 jwkvam

Confirm whether it's working now, cheers!

szymonmaszke avatar Feb 28 '18 22:02 szymonmaszke

Looks good to me thanks!

jwkvam avatar Feb 28 '18 22:02 jwkvam

Ah, that's why it didn't work!!

$> uname -a
Linux V 4.15.0-33-generic #36-Ubuntu SMP Wed Aug 15 16:00:05 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
$> notedown --version
1.5.1

I installed vimpyter via Vundle. I can't find the version via :help vimpyter-options. Do I have a version that is too old? How do I find out?

mrquincle avatar Sep 06 '18 12:09 mrquincle

Just update the plugin using :PluginUpdate and you should be fine. You can clone this repository and compare commits with the one located in Vundle, close if everything is fine.

szymonmaszke avatar Sep 06 '18 15:09 szymonmaszke

Ah, I only now discovered that Vundle is actually cloning your repository under the hood, neat. https://github.com/szymonmaszke/vimpyter.git. It's up to date with origin/master. So, that means there must still be some bugs somewhere. Can I somehow get you a log?

mrquincle avatar Sep 06 '18 15:09 mrquincle

Give me more details about your issues. Filename, does this plugin work for you at all? Vim/Neovim version, preferably directory where your .ipynb is located, anything you think could be related.

szymonmaszke avatar Sep 06 '18 19:09 szymonmaszke

This patch fixes it. For any file with spaces in the name, for example Keras Adversarial Autoencoder.ipynb only $TMPDIR/Keras will be created and will be empty. I narrowed it down to the system call to notedown where the target file apparently needs quotes. There might be other places in the code with filenames that actually need quotes as well, but this already fixes it on my system. Thanks for the plugin!!

mrquincle avatar Sep 06 '18 20:09 mrquincle