Dlux804 / McQuade-Chem-ML

Development of easy to use and reproducible ML scripts for chemistry.
5 stars 1 forks source link

Added doc strings and comments throughout core/files. #25

Closed Dlux804 closed 4 years ago

Dlux804 commented 4 years ago

Closes #9 issue.

pep8speaks commented 4 years ago

Hello @Dlux804! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 100:80: E128 continuation line under-indented for visual indent Line 100:80: E501 line too long (139 > 79 characters)

Line 22:80: E501 line too long (81 > 79 characters) Line 55:80: E501 line too long (81 > 79 characters) Line 57:80: E501 line too long (87 > 79 characters) Line 61:80: E501 line too long (83 > 79 characters) Line 85:80: E501 line too long (81 > 79 characters) Line 92:80: E501 line too long (156 > 79 characters) Line 124:80: E501 line too long (81 > 79 characters) Line 168:80: E501 line too long (81 > 79 characters) Line 183:32: E231 missing whitespace after ',' Line 241:80: E501 line too long (106 > 79 characters)

Line 2:80: E501 line too long (83 > 79 characters) Line 17:80: E501 line too long (86 > 79 characters) Line 24:80: E501 line too long (93 > 79 characters)

Line 42:80: E501 line too long (89 > 79 characters) Line 43:80: E501 line too long (102 > 79 characters)

Line 21:49: E231 missing whitespace after ',' Line 21:80: E501 line too long (85 > 79 characters) Line 24:30: E231 missing whitespace after ',' Line 24:37: E231 missing whitespace after ',' Line 24:44: E231 missing whitespace after ',' Line 24:51: E231 missing whitespace after ',' Line 24:58: E231 missing whitespace after ',' Line 24:80: E501 line too long (86 > 79 characters) Line 70:1: W293 blank line contains whitespace

Comment last updated at 2020-01-24 20:00:11 UTC
Dlux804 commented 4 years ago

Skipping review process. Functionality not changed, just cleaned up code and added docs and comments.