search

LLNL / merlin

Machine Learning for HPC Workflows
MIT License
122 stars 26 forks source link

Fix pylint errors #403

Closed bgunnar5 closed 1 year ago

bgunnar5 commented 1 year ago

This branch fixes almost all of the pylint errors we were seeing from the merlin/ and tests/ directory. I had to update the setup.cfg file to include some global disable flags for pylint.

lucpeterson commented 1 year ago

it looks like there are some merge issues that are causing CI to fail (search for <<<<)

lucpeterson commented 1 year ago

This is a big PR! It'll take me some time to parse through it

bgunnar5 commented 1 year ago

@lucpeterson A lot of the changes are just to fix pylint errors throughout merlin so most of the files should be quick to review luckily. Above I listed the files that have the more important changes. No rush on this PR, though :)