search

payu-org / payu

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

Added capability to specify hash functions for manifest #235

Closed aidanheerdegen closed 4 years ago

aidanheerdegen commented 4 years ago

Added tests for specifying hashes.

Fixed bug with test/models/test_mitgcm.py that globally changed config. Now specify config to write and make deep copies where appropriate.

Changed common input in tests to make local import.

Fixes #233

pep8speaks commented 4 years ago

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

Line 418:57: E127 continuation line over-indented for visual indent

Line 12:80: E501 line too long (109 > 79 characters) Line 12:82: E231 missing whitespace after ',' Line 12:91: E231 missing whitespace after ',' Line 14:1: E402 module level import not at top of file Line 15:1: E402 module level import not at top of file Line 16:1: E402 module level import not at top of file Line 17:1: E402 module level import not at top of file Line 18:1: E402 module level import not at top of file Line 19:1: E402 module level import not at top of file

Comment last updated at 2020-03-06 06:44:54 UTC