spinal-cord-7t / coil-qc-code

7T Spinal Cord Coil QC Analysis Code
0 stars 1 forks source link

Figure out notebook pre-commit hooks vs. post-commit stripping #140

Open jcohenadad opened 3 hours ago

jcohenadad commented 3 hours ago

Oops! I became lazy to remove my notebook outputs before committing, thinking that GHA would take care of it, but I guess the difference with a pre-commit hook, is that the commit with the notebook outputs becomes part of the git history, and then a bot strips it out and add another commit to it:

image

This is pretty annoying. I guess I should be looking at pre-commit hook approach, unless there is another solution?

jcohenadad commented 3 hours ago

In fact, the auto-strip commit is annoying because it systematically creates a conflict on the local branch:

image

We should probably get rid of it...

mathieuboudreau commented 2 hours ago

Maybe this? https://zhauniarovich.com/post/2020/2020-06-clearing-jupyter-output/