payu-org / payu

A workflow management tool for numerical models on the NCI computing systems
Apache License 2.0
21 stars 27 forks source link

Allow generic tracers with `cesm_cmeps` driver #433

Closed dougiesquire closed 6 months ago

dougiesquire commented 8 months ago

This is a ~work-in-progress~ PR with changes required to allow running generic_tracers in MOM with the cesm_cmeps driver. ~This PR is still under development and is likely to change. I'm only opening now for visibility and to help me keep track of the various changes I'm having to make across multiple repos to allow this functionality.~

Changes made:

pep8speaks commented 8 months ago

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

Line 120:80: E501 line too long (90 > 79 characters) Line 126:80: E501 line too long (85 > 79 characters) Line 307:80: E501 line too long (85 > 79 characters) Line 321:80: E501 line too long (89 > 79 characters) Line 324:80: E501 line too long (93 > 79 characters) Line 329:80: E501 line too long (87 > 79 characters) Line 335:1: W391 blank line at end of file

Line 6:1: E302 expected 2 blank lines, found 1 Line 13:51: E261 at least two spaces before inline comment Line 13:80: E501 line too long (87 > 79 characters) Line 14:51: E261 at least two spaces before inline comment Line 14:80: E501 line too long (88 > 79 characters) Line 15:52: E261 at least two spaces before inline comment Line 16:50: E261 at least two spaces before inline comment Line 16:80: E501 line too long (82 > 79 characters) Line 17:80: E501 line too long (148 > 79 characters) Line 17:105: E261 at least two spaces before inline comment Line 27:1: E302 expected 2 blank lines, found 1 Line 32:80: E501 line too long (90 > 79 characters) Line 34:1: E302 expected 2 blank lines, found 1 Line 50:1: E302 expected 2 blank lines, found 1 Line 58:9: E125 continuation line with same indent as next logical line Line 62:1: E302 expected 2 blank lines, found 1

Comment last updated at 2024-05-22 05:19:55 UTC