compgeolab/eql-gradient-boosted

Keep .ipynb files in the history as well

leouieda opened this issue · 1 comments

Jupytext is awesome to actually get a diff of the notebooks. But it also kind of sucks when I just want to see your results without having to run the code here (for example, it might take a long time). One alternative would be keep both the .py and .ipynb files in the history. We can review in PRs using the .py but be able to quickly check the outputs in the notebook. It's a bit wasteful but it's not like we'll be commiting to this repo for years.

What do you think?

I was planning to add the notebooks just before submission, in order to prevent the repo to get too cluttered with the embedded images on the .ipynb files. But I agree that even the .py files are easier to review, the lack of results on the notebooks is inconvenient, and saving each image is too tedious.

So, I agree to commit also the notebooks.