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
New Thompson cloud fraction (updated subroutine cal_cldfra3) #781
New Thompson cloud fraction (updated subroutine cal_cldfra3) #781
Changes from all commits
29db559
052145c
8da0705
3787899
f9c785f
4742485
cba377d
30ae919
74cae0c
fe6677f
25d28e7
b4d7ab0
aba5331
24d850d
6f91aea
211a413
0a54eec
1120779
7e119fc
bc8c26d
2b2e3b1
7d0d736
cd00e16
f1dd327
472987f
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.
Since both of us removed the lines of code above here related to icloud=3, then I think you should remove the next 2 chunks that are handling only the
lyb
layer as equal tolya
layer. I no longer think those lines are relevant since there is nothing inside the cldcov array at this point. That is why I also removed it.If I am incorrect, then my PR needs changing to put these back. Does something prior to this subroutine already pre-popluate those arrays?