-
Notifications
You must be signed in to change notification settings - Fork 134
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
MDF/MOSAiC forcing options #500
Draft
davidclemenssewall
wants to merge
45
commits into
CICE-Consortium:main
Choose a base branch
from
davidclemenssewall:mosaic_forcing_clean
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 34 commits
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
f61b4b1
up to reading time01
davidclemenssewall 8d0eb98
ran into ntime /= npt issue
davidclemenssewall 92b7cf3
made *_data arrays allocatable to solve ntime issue, now getting a se…
davidclemenssewall 52af4e1
forgot to allocate open_data and clos_data, fixed and solved seg fault
davidclemenssewall 4c9fed7
averaging function runs
davidclemenssewall 8c99a08
Now working for Tair
davidclemenssewall e9cac2d
works okay, qa/1000 and fsnow and frain set to 0
davidclemenssewall 3409f42
dummy ocean forcing and initial conditions closer to mosaic
davidclemenssewall 7055333
fixed istep /= istep1 issue, now can start at times other than Jan. 1
davidclemenssewall 312b24c
Merge branch 'main' into for_igs
davidclemenssewall c97bc56
updated forcing code now that MDF doesn't write calendar
davidclemenssewall c69f403
added functionality for precipitation and ocean mixed layer forcing
davidclemenssewall 71a176c
partially through changes to add ice thickness init options
davidclemenssewall 78f761c
init ice thickness complete and checked that results match
davidclemenssewall 9db2ef6
set initial sst working
davidclemenssewall 9f297c1
removed setting sst from icedrv_flux
davidclemenssewall f00f46b
fixed sss, qdp, and hmix all working
davidclemenssewall 5a6ae48
Merge branch 'main' into mosaic_ocn_forcing to stay up to date with c…
davidclemenssewall 6219577
Merge branch 'init_options' into mosaic_ocn_forcing
davidclemenssewall 397e09a
change atmmosaic options
davidclemenssewall dd92954
updated atmmosaic and ocnmosaic to new forcings
davidclemenssewall 2c8195a
updated names for downwelling radiation fields in forcing to correct …
davidclemenssewall e6c0dce
updated mosaic settings to account for fact that ADC converts - to _ …
davidclemenssewall 6be69b0
Merge branch 'main' into mosaic_forcing_clean
davidclemenssewall ac5ad88
Merge branch 'main' into mosaic_forcing_clean
davidclemenssewall c3a815d
Cleaned up extraneous comments and renamed precalc_forc
davidclemenssewall 946fbcb
added check for standard calendar and leap years
davidclemenssewall 38995e7
Merge branch 'main' into mosaic_forcing_clean
davidclemenssewall 0eb1066
changed driver such that sstdat is initialized from sst_init, instead…
davidclemenssewall 3c00c2f
reset sst initialization to align with main
davidclemenssewall ab1fd79
set qdp_fixed to 0.0 to align with ocn_data_type = 'default' and adde…
davidclemenssewall 12a692b
fixed trailing whitespace
davidclemenssewall 0b4b404
Merge branch 'main' into mosaic_forcing_clean
davidclemenssewall b47d4c8
added check to ensure that precalc_forc is only used for the MOSAiC f…
davidclemenssewall 7d88cbf
forcing refactoring and flexible cadence code complete, compiles, doe…
davidclemenssewall d93b3fc
fixed typo, now runs successfuly
davidclemenssewall b15d4d5
reorganized data format checks for clarity
davidclemenssewall 429e824
added warning for MOSAiC leg 4-5 transition
davidclemenssewall 358896d
added warning for impact of climatological atmosphere forcing on ocea…
davidclemenssewall 5f36f94
Changed forcing name from MOSAiC to MDF and updated check for MOSAiC …
davidclemenssewall 9cf540f
Updating documentation, MDF forcing added to ug_implementation, start…
davidclemenssewall c738251
partway through doc update
davidclemenssewall 8f9b081
Merge branch 'main' into mosaic_forcing_clean
davidclemenssewall 8ceed76
finished documentation update
davidclemenssewall fdafc83
updated documentation to include namelist options for initial ice sta…
davidclemenssewall 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
Oops, something went wrong.
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.
My preference would be to do this in a different PR, so its clear what the scope of the change is (i.e. its not related to MOSAIC forcing data)
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 totally agree that this would be best practice. But I admit that I'm loath to do so because the branch I originally made these modifications on is a bit stale and it doesn't feel like a productive use of time to sort through the merge conflicts and do all of the testing if the ultimate outcome for the code is the same. If others feel strongly about it I will try to get to it sometime next week. @apcraig @eclare108213 @dabail10 What do you think?
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 don't think it's worth the extra work at this point, to make a separate PR.