Revise code based on the recommends from Niklas
-
Recommends import future if you are in python3 version -
Use setup tool to set up the emails, authors, etc. But depends on your preferences -
Remove as metadata in the import -
Use fstring https://docs.python.org/3/tutorial/inputoutput.html#tut-f-strings to save the space -
line 186 : "Except" you catch all the errors, this will not be beneficial for the debugging. it is better to catch errors explicitly. -
Line 304: For Python 3, the range is the same for np.arange. -
847 - 861: you don't need to give a list (), keys() give you the iterators. -
884 - 886: Can be reduced to one line by using pandas, then pandas to NumPy … -
962-968: time_persistetnce.size == 0 to test the time persistent exmpty. -
Typos in the comments -
Comments format is not consistent. 1) https://www.sphinx-doc.org/en/master/ 2) Pycharm also handles the hover description wit the “:param bla:”-approach
Edited by Karim Mache