Pep8 non-compliance and minor bugs.

valeriupredoi / bgcval2

Package for BGCVal v2.0

3 stars 0 forks source link

Pep8 non-compliance and minor bugs. #106

Open ledm opened 11 months ago

ledm commented 11 months ago

Today, while working on PR #102, I've noticed several issues with bgcval2. Stuff like arguments that do nothing, things that are imported but never used, and some pep8 non-compliance. It would be nice to make it pretty.

Also, I've discovered the pycharm community python IDE today, so I might try to use that to generate some comments, and clean things up a bit.

Edit: Files that I've run over:

[ ] bgcval2_make_report.py
[ ] analysis_compare.py

Wider issues this has uncovered:

ledm commented 11 months ago

pycharm is finding so many ways to improve my old code. Lots of stuff like:

code alignment
incorrect capitals in file declarations
incorrect ordering of imports

ledm commented 11 months ago

Working on this here: https://github.com/valeriupredoi/bgcval2/tree/pep8_linting