Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
I care about |
mikemhenry
left a comment
There was a problem hiding this comment.
LGTM -- Just a question about the json file size
There was a problem hiding this comment.
Does it make sense to compress this json? it is 9.89 MB -- if we need it like this then we need it like this, its okay for the example notebooks to get bigger but it feels like a gzip on it would be worth it (also sorry if there are already a lot of these in the notebook)
There was a problem hiding this comment.
oh yikes, yeah this is larger than I realized and we really don't need it now that we have a whole septop analysis notebook. i'll comment it out like I did with the abfe notebook.
No description provided.