I notice a lot of commented out print statements, presumably from debugging the initial versions.  Do you really want to leave those in when merging it into master?  It looks kind of untidy.  (I didn't just remove them myself in case you were leaving them on purpose.)