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
CMAKE for WW3 #1089
CMAKE for WW3 #1089
Changes from 41 commits
18d6daa
02636e6
7a5ae43
365a2ee
033bf21
297221e
d07e9da
50f071b
348e770
706af74
e9984e2
0192be8
0ea8882
895a69f
4ecccd2
3f4baae
2ef38d2
8b1a94a
87b2756
ebafc42
c3e9b4d
557b4d6
cdaa26d
944fe7e
b2683df
99d7217
2d9fb80
d429a87
937995f
7a8c33b
dcb9c33
0128452
cd12a62
31e7e60
f0d0f36
ef0bf42
b0d016e
b47afcb
6793c2d
85a58ee
aeff388
a2732d4
45cf949
3f97a4b
149ecbb
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.
Is there a minimum cmake version required? Cheyenne.gnu/intel is loading cmake/3.18.2. Are changes required there?
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.
WW3 requires 3.19
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.
@DeniseWorthen is Cheyenne cmake/3.22.0 the correct version to switch to for the Cheyenne modules? (That was the version you mentioned on the WW3 PR, I forgot to update it here).
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.
Cheyenne has cmake/3.22.0. Thanks.
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.
Updated for Cheyenne
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 a thought, but it might be wise to update the
cmake_minimum_required
in UFS to 3.19.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.
@DusanJovic-NOAA Could you make this change in your PR (3.18->3.19):
ufs-weather-model/CMakeLists.txt
Line 1 in 10a616d
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.
Done.