-
Notifications
You must be signed in to change notification settings - Fork 150
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
cam6_4_023: SCAM-SE feature addition plus bugfixes and some refactoring. #958
Merged
Merged
Changes from 22 commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
872b60e
build script mods to allow se builds
jtruesdal 3006e9c
se scam update
jtruesdal 55ab657
bfb mods
jtruesdal da514d3
update to cam_development (cam6_3_128)
jtruesdal fd265ed
se scm updates to cmeps cdeps externals
jtruesdal f202756
scam updates
jtruesdal 1e56c28
update to cam6_3_136 need to update externals yet
jtruesdal e4f97ea
external updates for scam_dev 6_3_136
jtruesdal 64aa11e
missed cice external update for scam
jtruesdal b6ee97c
scam se update
jtruesdal 5a9159a
scam se updates
jtruesdal b4ed83a
scam clean and updates for derecho
jtruesdal b567e51
bug fix for iopTimeIdx when iop step size is smaller than model timestep
jtruesdal e7d33d7
PR updates for Cheryl and Jim
jtruesdal aac8cf6
new default initialization using config_compset and config_component,…
jtruesdal 6e9b777
check for error with mpi_bcast calls
jtruesdal 91af07d
PR requested updates
jtruesdal 53300de
compile catch/fix for se_single_column_mod
jtruesdal 36e8f4b
shorten a few long lines >132
jtruesdal 8851ffd
bugfix for new minloc replacement for loop with exit
jtruesdal 5f7a485
update ice external tag
jtruesdal bcd2ea5
dont need these cdep mods
jtruesdal 2e344b3
PR updates for Jesse
jtruesdal 0407135
PR updates
jtruesdal 551612d
PR updates
jtruesdal 6f988cd
PR updates
jtruesdal bfc7658
PR update for testing
jtruesdal 66b6a24
merge up to cam6_4_006
jtruesdal 6244ced
remove bfb line for testing
jtruesdal a972f69
merge up to cam6_4_018
jtruesdal dcc636d
PR updates: comments,typo, and replace hard coded length with shr_kin…
jtruesdal e004f55
Merge scam_dev_exp up to cam6_4_022
jtruesdal 54a0ba9
update cice and cdeps for regression failures,Changelog for PR,scam t…
jtruesdal b88c67e
need scam_mandatory to set COLDSTART until CLM scam mods tagged
jtruesdal 40f6c5c
update mpace regression test and ChangeLog
jtruesdal c50362c
update ChangeLog, pull hvcoord addition
jtruesdal 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 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 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 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 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
Oops, something went wrong.
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.
Why did you change to a fork if you otherwise have no mods 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.
i'll double check this and get rid of it. 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.
Tag name should be share1.0.17_scamdev. I had a few mods for shr_scam_mod that turned up in testing for this PR. If you can help with the mods for getting rid of MCT I'm happy to include them, test and create the PR to CESM_share. In the meantime I'll fix the tag reference above.
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.
@jedwards4b forgot to 'at' you on the previous question. Repeating 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.
Sorry I haven't looked into it yet, I'll try to do it tomorrow.