Dlux804 / McQuade-Chem-ML

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

Organize #19

Closed Dlux804 closed 4 years ago

Dlux804 commented 4 years ago

File organization is completed.

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 135:80: E501 line too long (85 > 79 characters) Line 144:1: E303 too many blank lines (4)

Line 20:80: E501 line too long (87 > 79 characters) Line 24:80: E501 line too long (88 > 79 characters)

Line 2:10: E401 multiple imports on one line Line 12:1: E402 module level import not at top of file

Line 3:10: E401 multiple imports on one line

Comment last updated at 2020-01-15 15:41:06 UTC
Dlux804 commented 4 years ago

@andreshyer I have fixed the things keeping this PR from passing unit tests. It is ready for your review.

Dlux804 commented 4 years ago

@qle2 This branch will merge to master soon (feel free to review and merge it). It has reorganized many of the files and that affected some of the testing files. Please make sure to update your testing branch with these updates.