Fates exercise#359
Merged
Merged
Conversation
Collaborator
|
Hi @adrifoster. Thanks for these. It does look like you have a few conflicts that need to be resolved before we can merge this. Let me know if you need help. |
ekluzek
reviewed
Jul 9, 2026
Contributor
Author
|
@dabail10 I believe this is ready now?? |
ekluzek
reviewed
Jul 9, 2026
ekluzek
left a comment
ekluzek
left a comment
There was a problem hiding this comment.
I have a couple suggestions to make that are simple.
| "- FATES: The FATES cohort vegetation model instead of big leaf, either SP or BGC\n", | ||
| "\n", | ||
| "We will use the Satellite Phenology option for our control case." | ||
| "We will use the Satellite Phenology option for our first simulation." |
There was a problem hiding this comment.
I was thinking "control case" is good to use, because then it's the one that the others are compared to. What is the thinking about this change?
| "metadata": {}, | ||
| "source": [ | ||
| "We can use a different I compset: IHistClm50BgcCrop. This experiment is a 20th century transient run using GSWP3v1 and the biogeochemistry model including crops." | ||
| "We can use a different I compset: IHistClm60BgcCrop. This experiment is a 20th century transient run using CRUJRA and the biogeochemistry model including crops.\n", |
There was a problem hiding this comment.
Suggested change
| "We can use a different I compset: IHistClm60BgcCrop. This experiment is a 20th century transient run using CRUJRA and the biogeochemistry model including crops.\n", | |
| "We can use a different I compset: IHistClm60BgcCrop. This experiment is a 20th century transient run using GSWP3 forcing and the biogeochemistry model including crops.\n", |
| "- You will run a control simulation and two experimental simulations. \n", | ||
| "- You will modify netcdf input files. \n", | ||
| "- You will use simple, command line netcdf tools to evaluate how the experiments differ from the control simulation." | ||
| "- You will be using the I2000Clm60Sp, IHistClm60BgcCrop, and I2000Clm60FatesSpCrujraRsGs compsets at the f19_f19_mt233 resolution.\n", |
There was a problem hiding this comment.
I think we should add a note that the forcing changes between these three.
There was a problem hiding this comment.
Suggested change
| "- You will be using the I2000Clm60Sp, IHistClm60BgcCrop, and I2000Clm60FatesSpCrujraRsGs compsets at the f19_f19_mt233 resolution.\n", | |
| "- You will be using the I2000Clm60Sp, IHistClm60BgcCrop, and I2000Clm60FatesSpCrujraRsGs compsets at the f19_f19_mt233 resolution.\n", | |
| "- The first two use GSWP3 atmospheric forcing and the last uses CRUJRA-2024 forcing.\n", |
|
Oh, never mind, it's already merged. I'll do those seperately. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
I added some FATES exercises.