Implement --remove-tagged-cells

[astrofrog]

This implements a new --remote-tagged-cells option as described by @MSeal in https://github.com/nteract/papermill/issues/429#issuecomment-642806948. The idea of the current implementation is to keep it as simple as possible – only being able to specify a single tag and requiring an exact match (no regular expressions etc). My personal use case for this is being able to run notebooks with/without diagnostic plotting but there are other use cases described in https://github.com/nteract/papermill/issues/429.

If the maintainers are on board with this PR so far, I can add documentation and anything else required.

One thought I had was that perhaps this option would read more clearly as --remove-cells-with-tag otherwise it sounds like a boolean option, and also doesn't make it clear that a single tag is what should be passed in. What do people think about this?

[review-notebook-app[bot]]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[willingc]

Thanks @astrofrog (nice to see you here). It looks like some of the CLI tests still need updating. It also looks like botocore is being grumpy with python 3.12.

[MSeal]

Thanks @astrofrog -- on the botocore build part I am working on getting a new release out anyway and need to fix dep updates since the last one regardless.

[MSeal]

Looks like tests are failing now due to some test scaffolding including the input_path in the arguments.

[akx]

Is there something I could do to move this PR forward, having been the instigator of #429 back in 2019? 😁 (We've used a custom fork of Papermill in the meantime, but it's fallen quite out of repair, so it'd be nice to use the upstream version.)

[andreytaboola]

Any updates on this? I am also interested in this feature and will appreciate push on it Thank you