search

Dlux804 / McQuade-Chem-ML

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

Development andres #56

Closed andreshyer closed 4 years ago

andreshyer commented 4 years ago

Major updated to models.py. No change in functionality in core functions, except files (such as regressors.py) can no longer directly modify a model object. Added a function to storage.py to pickle and unpickle class object. And added example_model to main.py to show how structure works

pep8speaks commented 4 years ago

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

Line 6:80: E501 line too long (116 > 79 characters)

Line 70:1: E302 expected 2 blank lines, found 1 Line 70:46: E251 unexpected spaces around keyword / parameter equals Line 70:48: E251 unexpected spaces around keyword / parameter equals Line 113:80: E501 line too long (124 > 79 characters) Line 114:80: E501 line too long (123 > 79 characters) Line 115:80: E501 line too long (133 > 79 characters) Line 116:80: E501 line too long (135 > 79 characters) Line 128:80: E501 line too long (127 > 79 characters) Line 129:80: E501 line too long (138 > 79 characters) Line 130:80: E501 line too long (137 > 79 characters) Line 140:9: E303 too many blank lines (2)

Line 54:80: E501 line too long (97 > 79 characters) Line 55:80: E501 line too long (84 > 79 characters) Line 56:80: E501 line too long (96 > 79 characters) Line 134:80: E501 line too long (83 > 79 characters) Line 137:80: E501 line too long (82 > 79 characters) Line 169:80: E501 line too long (112 > 79 characters)

Line 13:80: E501 line too long (92 > 79 characters) Line 29:80: E501 line too long (106 > 79 characters) Line 73:80: E501 line too long (112 > 79 characters) Line 88:80: E501 line too long (96 > 79 characters) Line 89:80: E501 line too long (101 > 79 characters) Line 103:80: E501 line too long (80 > 79 characters) Line 112:5: E303 too many blank lines (2) Line 120:80: E501 line too long (87 > 79 characters) Line 128:80: E501 line too long (106 > 79 characters) Line 143:44: W605 invalid escape sequence '\o' Line 152:80: E501 line too long (87 > 79 characters) Line 154:80: E501 line too long (97 > 79 characters) Line 158:80: E501 line too long (87 > 79 characters) Line 163:80: E501 line too long (109 > 79 characters) Line 164:80: E501 line too long (105 > 79 characters) Line 165:80: E501 line too long (92 > 79 characters) Line 167:80: E501 line too long (103 > 79 characters) Line 168:80: E501 line too long (98 > 79 characters) Line 169:80: E501 line too long (93 > 79 characters) Line 172:80: E501 line too long (91 > 79 characters) Line 176:80: E501 line too long (94 > 79 characters) Line 182:80: E501 line too long (85 > 79 characters) Line 186:80: E501 line too long (93 > 79 characters) Line 200:80: E501 line too long (90 > 79 characters) Line 201:80: E501 line too long (88 > 79 characters)

Line 21:80: E501 line too long (82 > 79 characters) Line 131:80: E501 line too long (109 > 79 characters) Line 132:80: E501 line too long (117 > 79 characters) Line 146:80: E501 line too long (109 > 79 characters)

Line 9:80: E501 line too long (92 > 79 characters) Line 61:80: E501 line too long (91 > 79 characters) Line 76:45: E261 at least two spaces before inline comment Line 103:80: E501 line too long (96 > 79 characters) Line 104:80: E501 line too long (92 > 79 characters) Line 108:80: E501 line too long (98 > 79 characters) Line 112:80: E501 line too long (102 > 79 characters) Line 113:26: E128 continuation line under-indented for visual indent Line 126:80: E501 line too long (84 > 79 characters) Line 127:80: E501 line too long (82 > 79 characters) Line 130:80: E501 line too long (109 > 79 characters)

Comment last updated at 2020-06-17 16:50:57 UTC