We ran 2 automated code analysis then we added a series of TODOS for potential errors/issues: Please review all TODO comments we left in the code and address them if necessary. We used both [Anaconda Linting] (http://damnwidget.github.io/anaconda/IDE/) and PyCharm code inspection.
Adjusted style wherever necessary to comply with the PEP8 style guide.
Adopted python package/module standards:
Fixed hard coded path to prob_model_path:
Fixed Mutable Default Arguments, which is a common "Python Gotcha" that leads to unexpected behaviour. Please refer to the topic Mutable Default Arguments in The Hitchiker's Guide to Python website.
We ran 2 automated code analysis then we added a series of TODOS for potential errors/issues: Please review all TODO comments we left in the code and address them if necessary. We used both [Anaconda Linting] (http://damnwidget.github.io/anaconda/IDE/) and PyCharm code inspection.
Adjusted style wherever necessary to comply with the PEP8 style guide.
Others: