Logic sound good for me.
Comments: L56: could you add "summary" with a short description L84,L98: you would be better off turning these comments into docstrings
« Back to merge proposal
Logic sound good for me.
Comments:
L56: could you add "summary" with a short description
L84,L98: you would be better off turning these comments into docstrings