Dlux804 / McQuade-Chem-ML

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

Update to fix param_grid export to json #58

Closed andreshyer closed 4 years ago

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 19:80: E501 line too long (100 > 79 characters) Line 60:80: E501 line too long (119 > 79 characters)

Line 28:80: E501 line too long (81 > 79 characters) Line 29:80: E501 line too long (105 > 79 characters) Line 42:80: E501 line too long (81 > 79 characters) Line 43:80: E501 line too long (101 > 79 characters) Line 82:80: E501 line too long (81 > 79 characters) Line 122:80: E501 line too long (96 > 79 characters)

Comment last updated at 2020-06-18 15:49:55 UTC
Dlux804 commented 4 years ago

@andreshyer Please update this one with description too.