Minor fix to find_tempo2_runtime

vallis / libstempo

libstempo — a Python wrapper for tempo2

MIT License

29 stars 34 forks source link

Minor fix to find_tempo2_runtime #45

Closed mattpitkin closed 2 years ago

mattpitkin commented 2 years ago

Currently, find_tempo2_runtime first checks a path within your home directory for the tempo2 runtime location, assuming the the HOME variable is a value. However, if running libstempo on a cluster node it is not necessarily the case that HOME is set, which means that it could be None. In this case the Path(HOME) command will fail with a TypeError. This minor fix only checks the local path is HOME is not a None-type.

vallis commented 2 years ago

Thanks Matt! As a matter of process, if you merge mattpitkin:black (now merged) or vallis:master into this branch, then the tests should succeed and then I can merge this.

mattpitkin commented 2 years ago

Thanks. I've now merged the black fixes into this MR.