Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Collect timeslices #187

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Conversation

jmccreight
Copy link
Collaborator

Code, tests, and updated test files to support the collection of timeslice files to a single netcdf file.

@pep8speaks
Copy link

pep8speaks commented Jul 22, 2019

Hello @jmccreight! Thanks for updating this PR.

Line 295:101: E501 line too long (101 > 100 characters)

Line 4:101: E501 line too long (1191 > 100 characters)
Line 6:101: E501 line too long (779 > 100 characters)
Line 8:101: E501 line too long (850 > 100 characters)
Line 10:101: E501 line too long (600 > 100 characters)
Line 12:101: E501 line too long (966 > 100 characters)
Line 14:101: E501 line too long (8195 > 100 characters)
Line 16:101: E501 line too long (1554 > 100 characters)
Line 18:101: E501 line too long (5511 > 100 characters)
Line 20:101: E501 line too long (2669 > 100 characters)
Line 28:101: E501 line too long (1529 > 100 characters)
Line 30:101: E501 line too long (1099 > 100 characters)
Line 32:101: E501 line too long (1167 > 100 characters)
Line 34:101: E501 line too long (872 > 100 characters)
Line 36:101: E501 line too long (1268 > 100 characters)
Line 38:101: E501 line too long (9460 > 100 characters)
Line 40:101: E501 line too long (2083 > 100 characters)
Line 42:101: E501 line too long (7019 > 100 characters)
Line 44:101: E501 line too long (3238 > 100 characters)
Line 52:101: E501 line too long (1274 > 100 characters)
Line 54:101: E501 line too long (892 > 100 characters)
Line 56:101: E501 line too long (945 > 100 characters)
Line 58:101: E501 line too long (662 > 100 characters)
Line 59:101: E501 line too long (1069 > 100 characters)
Line 60:101: E501 line too long (8389 > 100 characters)
Line 61:101: E501 line too long (1703 > 100 characters)
Line 63:101: E501 line too long (2736 > 100 characters)
Line 65:101: E501 line too long (5910 > 100 characters)
Line 73:101: E501 line too long (1529 > 100 characters)
Line 75:101: E501 line too long (1099 > 100 characters)
Line 77:101: E501 line too long (1167 > 100 characters)
Line 79:101: E501 line too long (872 > 100 characters)
Line 81:101: E501 line too long (1268 > 100 characters)
Line 83:101: E501 line too long (9460 > 100 characters)
Line 85:101: E501 line too long (2083 > 100 characters)
Line 87:101: E501 line too long (7019 > 100 characters)
Line 89:101: E501 line too long (3238 > 100 characters)
Line 97:101: E501 line too long (1632 > 100 characters)
Line 99:101: E501 line too long (1176 > 100 characters)
Line 101:101: E501 line too long (1238 > 100 characters)
Line 103:101: E501 line too long (949 > 100 characters)
Line 105:101: E501 line too long (1361 > 100 characters)
Line 107:101: E501 line too long (10289 > 100 characters)
Line 109:101: E501 line too long (2248 > 100 characters)
Line 111:101: E501 line too long (3536 > 100 characters)
Line 113:101: E501 line too long (7567 > 100 characters)
Line 121:101: E501 line too long (1609 > 100 characters)
Line 123:101: E501 line too long (7276 > 100 characters)
Line 125:101: E501 line too long (3375 > 100 characters)
Line 133:101: E501 line too long (1314 > 100 characters)
Line 140:101: E501 line too long (1587 > 100 characters)
Line 142:101: E501 line too long (7564 > 100 characters)
Line 144:101: E501 line too long (3536 > 100 characters)
Line 152:101: E501 line too long (1448 > 100 characters)
Line 154:101: E501 line too long (6917 > 100 characters)
Line 156:101: E501 line too long (3247 > 100 characters)
Line 164:101: E501 line too long (558 > 100 characters)
Line 166:101: E501 line too long (556 > 100 characters)

Line 9:18: W291 trailing whitespace
Line 16:6: W291 trailing whitespace
Line 32:1: W293 blank line contains whitespace

Comment last updated at 2019-09-18 21:42:17 UTC

@coveralls
Copy link

Pull Request Test Coverage Report for Build 432

  • 44 of 44 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.1%) to 89.736%

Totals Coverage Status
Change from base Build 430: 0.1%
Covered Lines: 3025
Relevant Lines: 3371

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants