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

Understimation of incoming longwave radiation when forcing the model with downscaled shortwave radiation #52

Closed
homefc opened this issue Sep 9, 2013 · 3 comments
Assignees
Milestone

Comments

@homefc
Copy link
Contributor

homefc commented Sep 9, 2013

When downscaled daily downward shortwave radiation is used as an input to the model, for some reason incoming longwave is calculated assuming clear skies always, and this results in an underestimate of incoming longwave on cloudy days.
The error is likely linked to the estimate of the clear sky fraction in MTCLIM.

@ghost ghost assigned tbohn Sep 11, 2013
@bartnijssen
Copy link
Member

OK - let's make sure that this gets tested and makes it in the new bug fix update (hot fix) as soon as possible. This will affect a number of ongoing projects and is important to get right.

Homero: Please suggest a fix and test it.

I have assigned Ted to do the actual merge, since he has mostly dealt with the forcing side of the business.

@tbohn
Copy link
Contributor

tbohn commented Sep 11, 2013

Yes, Homero mentioned this to me, and I am going to look into it soon.

On Wed, Sep 11, 2013 at 4:28 PM, bartnijssen notifications@github.comwrote:

OK - let's make sure that this gets tested and makes it in the new bug fix
update (hot fix) as soon as possible. This will affect a number of ongoing
projects and is important to get right.

Please suggest a fix and test it. I have assigned Ted, since he has mostly
dealt with the forcing side of the business.


Reply to this email directly or view it on GitHubhttps://github.com//issues/52#issuecomment-24284496
.

@homefc
Copy link
Contributor Author

homefc commented Sep 12, 2013

Bart, Ted, I think I've found the fix and will send a pull request after I have tested it, so that Ted can merge it.

@jhamman jhamman closed this as completed Sep 17, 2013
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

No branches or pull requests

4 participants