Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Refactor gridded wave post #3014
base: develop
Are you sure you want to change the base?
Refactor gridded wave post #3014
Changes from 31 commits
26e06a3
99dea4a
97b5932
64552f8
ed3768c
9373b88
916c1fa
f018d2e
aede2e9
f59a6c9
f07b0ec
6261b4f
b2b3078
ea023be
3c69e48
16205f1
0c7cb50
458b130
027e79f
f75bbf3
b408e75
d96631e
08c4f12
bc74e49
99869ed
8b9d5d1
2646773
f31c4c1
4f58676
bed6e67
e742c7d
bd34f6c
a0f6cfd
901a299
4fc487b
f9ed53e
55171ec
12619c9
b78ea08
232d438
f7c4ba4
2e4d298
1028319
4fe0e52
95d2532
bda6124
8cbffe5
50550e4
bcd3386
bad7250
8a0833b
4db4b82
96934ca
cb6881d
d378f80
636e44c
dc6049e
d279aa2
a9fb44d
a179f2b
88c4cce
2446ff3
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This section can just be removed. The forecast hour is provided and does not need to be calculated anymore.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would just take this out. It is up to the workflow to determine which hours to run for.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@WalterKolczynski-NOAA All contents inside the loop? L239 - L432 or just the loop?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just the
if
that replaces thewhile
Still need most of the stuff inside it.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't delete it. Just commented it out.