Closed fu5ha closed 4 years ago
Awesome, thanks for doing all of that! I will have a read through over the weekend. I can look at the summary script.
Just realised I'd made a bunch of comments yesterday but forgot to hit the start review button...
How are you getting on with this @termhn ? I realize you've already put a heap of work into this, so I could always merge it over into a branch and address the remaining PR issues myself if you'd prefer.
@bitshifter Been a bit busy but I should be able to get around to fixing these up soon (maybe today, not sure)
Alright, @bitshifter I think this is ready for further review :) I think I addressed most of the points, and also rebased on master.
Awesome, thank you. I will take a look soon.
Thanks again. There's a few small issues but I can sort them out.
Note that I haven't tried the modifications I made to the summary script because I don't have a python env setup on my main machine atm.