Dlux804 / McQuade-Chem-ML

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

Q dev #89

Closed qle2 closed 4 years ago

qle2 commented 4 years ago

This PR mainly addresses:

-Add training data to the ontology

pep8speaks commented 4 years ago

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

Line 199:57: E127 continuation line over-indented for visual indent Line 199:80: E501 line too long (119 > 79 characters) Line 200:80: E501 line too long (97 > 79 characters) Line 201:80: E501 line too long (104 > 79 characters) Line 202:80: E501 line too long (107 > 79 characters) Line 203:80: E501 line too long (89 > 79 characters) Line 232:80: E501 line too long (92 > 79 characters)

Line 32:80: E501 line too long (100 > 79 characters)

Line 440:80: E501 line too long (85 > 79 characters) Line 441:80: E501 line too long (85 > 79 characters) Line 448:80: E501 line too long (92 > 79 characters)

Line 2:80: E501 line too long (96 > 79 characters)

Line 268:80: E501 line too long (120 > 79 characters) Line 291:80: E501 line too long (93 > 79 characters)

Line 66:80: E501 line too long (88 > 79 characters) Line 107:80: E501 line too long (107 > 79 characters) Line 125:80: E501 line too long (98 > 79 characters)

Line 112:80: E501 line too long (115 > 79 characters) Line 115:80: E501 line too long (111 > 79 characters)

Line 21:1: W391 blank line at end of file

Comment last updated at 2020-09-09 20:52:20 UTC
andreshyer commented 4 years ago

The branch looks good to me, make sure you figure out why the circleci tests are failing before merging