Create ClearMetadata Preprocessor #805
Conversation
|
|
||
| class ClearMetadataPreprocessor(Preprocessor): | ||
| """ | ||
| Removes the output from all code cells in a notebook. |
There was a problem hiding this comment.
This docstring seems to be copied and pasted — could you update it to reflect your intention for the class?
|
|
||
| def preprocess_cell(self, cell, resources, cell_index): | ||
| """ | ||
| Apply a transformation on each cell. See base.py for details. |
There was a problem hiding this comment.
This docstring is not specific to the purpose you are aiming at here, could you make it more specific.
| if cell.cell_type == 'code': | ||
| # Remove metadata | ||
| if 'metadata' in cell: | ||
| cell.metadata = {} |
There was a problem hiding this comment.
@takluyver we've had lots of issues about this — I actually think this is going to be one of the cleanest ways to implement this since it doesn't have exceptions or caveats. Just cleans everything (which from a version control perspective is what you want).
what do you think?
There was a problem hiding this comment.
Version control was the purpose of develop this. With this feature it will be possible to apply a clean and smudge for notebooks.
|
Docstrings updated |
|
@mpacer Are the docstrings good now? |
|
@mpacer any news about this? Thanks |
|
@mpacer I'm good with merging this. Looks simple and straight forward. Any reasons not to add it? |
Very simple preprocessor that cleans all the metadata on the ipynb code cells.
This is good for apply a git filter in order to be able to git diff more easily.