should this use ```os.path.join()``` ?