There was a problem where Bellhop was not correctly detected in the _bellhop function. More specifically, it shows up as a possible model even if it is not on the path. To reproduce, ensure Bellhop is not in your path and then run uwapm.models(). Bellhop appears in the list even though it is not on the path. Here is a demonstration:
The problem is that the return code of the subprocess command needs to be checked. I added code to check the return code and added a test.
There was a problem where Bellhop was not correctly detected in the
_bellhopfunction. More specifically, it shows up as a possible model even if it is not on the path. To reproduce, ensure Bellhop is not in your path and then runuwapm.models(). Bellhop appears in the list even though it is not on the path. Here is a demonstration:The problem is that the return code of the subprocess command needs to be checked. I added code to check the return code and added a test.