it probably doesn't matter but i think this will create a duplicate filedir entry on the path? you could have just used `sys.path.insert(1, submodule_path)`