TDA-tutorial
TDA-tutorial copied to clipboard
GUDHI
Use some tools to diff notebooks
Some tools like https://www.reviewnb.com/ may be helpful for the reviews
I installed reviewnb only for GUDHI/TDA-tutorial.
Now, if you want to review https://github.com/GUDHI/TDA-tutorial/pull/25 for instance, you can go to https://app.reviewnb.com/GUDHI/TDA-tutorial/pull/25/ and in changes, you select an *.ipynb file and you will see modifications with a diff tool that renders the notebook (instead of diff on raw notebooks files). It also works for commits, which is quite usefull.
Your feedbacks are welcome.
That looks very nice to me, thank you. Would it be easy to have a link to the corresponding reviewnb appear magically on each PR, or should I keep a bookmark(let) locally for when I need it?
That's pretty neat, thanks @VincentRouvreau. I am wondering whether we can add comments directly in reviewnb. What I mean is that when reviewing code in the Github interface, you can select part of the code and put a comment for that specific selection. Would that be possible to do the same thing while skimming through the changes using reviewnb?
Yes, it is possible to comment, I did it in #31 to see what it looked like.