ansys / ansys-tools-path

Library to detect Ansys products installation path
https://path.tools.docs.pyansys.com/
MIT License
3 stars 2 forks source link

fix: add an exception if executable not found #184

Closed dipinknair closed 2 months ago

dipinknair commented 3 months ago

Currently, if no executable found in default location when using get_mechanical_path , then we are returning None. This is causing issue when we call version_from_path where we give path as input. We do not have any check if path is None.

Alternately we can avoid this exception and return path as empty string (as it is) it is instead of None. If that is the case then version_from_path will take care of value error if path is empty string

RobPasMue commented 2 months ago

We should do a new release after #193 - thanks @nmalsang - what's the status here? We could include this PR as well.