Skip to content
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

Skip using fluxes provided by land component for first time step #234

Open
wants to merge 1 commit into
base: ufs/dev
Choose a base branch
from

Conversation

uturuncoglu
Copy link
Collaborator

Due to the dependency between atmosphere and land components, the atmosphere model gets all zero from land component model in the initial coupling time step. This PR aims to fix this issue and skips data coming from land component (all zero) at first coupling time step and use the land fluxes calculated by noahmp land model found in ccpp/physics. After first coupling time step, the coupled configuration (cpld_control_p8_lnd) will continue to use fluxes coming from component model.

@uturuncoglu
Copy link
Collaborator Author

JFYI, The top level UFS WM PR is still in draft mode. I am still working on it.

@uturuncoglu
Copy link
Collaborator Author

@grantfirl JFYI, I might also need to introduce another input.nml option to make skipping first time step for land optional since it is related with the run sequence of the coupled application and we might not need for the data atmosphere coupled configurations.

@grantfirl
Copy link
Collaborator

@grantfirl JFYI, I might also need to introduce another input.nml option to make skipping first time step for land optional since it is related with the run sequence of the coupled application and we might not need for the data atmosphere coupled configurations.

@uturuncoglu Thanks. I see that the upstream PRs are in draft. We'll re-review if necessary. There are several PRs to be merged before this one, so it's unlikely this would bubble to the top in 2 weeks time anyway.

@uturuncoglu
Copy link
Collaborator Author

@grantfirl Thanks. That is totally fine. I am still working on it. Once it is ready I'll change UFS WM level PR from draft to ready to review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants