-
Notifications
You must be signed in to change notification settings - Fork 2
Use develop branches for repositories in ufs bundle #30
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
Merged
Merged
Changes from all commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
0c9c122
update hashes
cmgas 7ac8bcb
Update CMakeLists.txt
cmgas 42d6139
update apr 18
cmgas beec989
build crtm
cmgas 84e49a8
gitignore update
cmgas 0c6f277
Update CMakeLists.txt
cmgas f877646
correct branches
cmgas 675edfc
build with develop branches
cmgas 6d7db8e
3dfgat branches
cmgas 86477d3
develop for oops
cmgas 485e4a0
use develop branches
cmgas f43c779
modify branch in cmakelist
cmgas 9e4a576
Update README.md
climbfuji 74077d9
Revert to feature/ufs_dom for fv3-jedi
climbfuji 82af153
Update bug fix for libpiof.dylib in README.md
climbfuji c510b93
Update CMakeLists.txt - use correct eckit version
climbfuji 191d326
Update CMakeLists.txt - use correct fckit version
climbfuji 296bdad
Update CMakeLists.txt - use correct atlas version
climbfuji 9ef55a0
Update CMakeLists.txt - use CRTM v3
climbfuji 243cb40
Merge branch 'feature/use_develop' of https://github.com/jcsda/ufs-bu…
climbfuji 0f1c777
Merge branch 'develop' of https://github.com/jcsda/ufs-bundle into fe…
climbfuji 8348529
Revert soca branch to feature/ufs_dom
climbfuji File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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.
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.
Does this include the change that soca needed?
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.
Yes!
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.
@travissluka tested this already (he said)
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.
🎉 you rock!
we'll test with fv3-jedi too, unless @travissluka you already tested with both soca and fv3-jedi?
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.
Well, the tag of fms contains the changes that SOCA needs, but the ufs-weather-model, fv3-jedi-linearmodel and fv3-jedi don't work with that tag ye. See ufs-community/ccpp-physics#90 and https://github.com/JCSDA-internal/fv3-jedi-linearmodel/issues/18 for examples. Also, see NOAA-GFDL/FMS#1284 for bugs in the build system of that tag.
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.
Got it, so testing https://github.com/JCSDA-internal/fv3-bundle/pull/103 (with updates to jcsda repos) isn't expected to work?
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.
At least at the moment, not. Let's see how far we get this week.