Piping not working on Python 3.5 about nbstripout HOT 13 CLOSED

However you can just use python nbstripout.py ... if you want to test the version from your Git checkout.

Right, but in this case the bug only affected the nbstripout script.

Thanks for the fix, by the way. Pleasure working with you.

from nbstripout.

I can't reproduce this. Have tried with both my system Python 3 and Anaconda Python 3. Could it be an issue with your terminal? Which platform are you on and which terminal are you using?

from nbstripout.

Interesting. I'm using NixOS so that definitely can be a source of issues, though I don't know what it could be in this case. I'm using Bash on Konsole. I'd expect it is related to something that is done differently on Python 2 than on Python 3. Any ideas are welcome.

from nbstripout.

And even more weird, this works on Python 3:

cat test_metadata.ipynb | python -m nbstripout

So it's just when it's used as a standalone executable in Python 3 that it doesn't handle piping properly. This is probably related to how NixOS wraps executables.

from nbstripout.

OK, good to know. Report back if you find out anything more.

from nbstripout.

I can reproduce this on my Mac:

$ nbstripout < test_metadata.ipynb | wc
       0       0       0
$ python -m nbstripout < test_metadata.ipynb | wc
      88     167    1556

The problem appears to be buffering related. On one of my own notebooks, it appears that the output is empty unless it exceeds roughly 8192 bytes.

Adding output_stream.flush() after line 279 fixes the problem for me.

Version 0.3.0. Installed using conda.

from nbstripout.

@reidpr Thanks! I've pushed that fix. Can you try with master? I want to tag a new release.

from nbstripout.

Happy to. Can you give me a brief recipe to get the executable nbstripout in a Git working directory? python setup.py build doesn't seem to make it, and python setup.py test complains pkg_resources.DistributionNotFound: The 'pytest' distribution was not found and is required by the application.

from nbstripout.

The 2 simplest options are:

• pip install git+https://github.com/kynan/nbstripout from anywhere
• python setup.py develop from a clone of this repo

from nbstripout.

@reidpr your fix will be released in 0.3.1

from nbstripout.

@jluttine @reidpr can you please test 0.3.1 and report if this is now fixed?

from nbstripout.

This seems to work in 0.3.1.

Sorry I didn't test it earlier. It seemed like the test instructions required installing the version under test, so I kept putting it off. It would be great if everything could be tested from a Git checkout, without installing it.

from nbstripout.

The nbstripout script is just a setuptools entrypoint. To use that you do indeed need to install it. I recommend a python setup.py develop / pip install -e .. I'll add that to the README.

However you can just use python nbstripout.py ... if you want to test the version from your Git checkout.

from nbstripout.