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

Prevent stale values of calculate_soil from leaking across threads #136

Open
wants to merge 1 commit into
base: release-v4.6-WRF
Choose a base branch
from

Conversation

lrbison
Copy link

@lrbison lrbison commented Jul 17, 2024

Addresses issue #135.

@lrbison
Copy link
Author

lrbison commented Jul 17, 2024

Another solution would be to move calculate_soil to be local to the noahmplsm subroutine, but that seems like it would have farther-reaching impact. Since all threads are on the same timestep, they should all end up with the same value of calculate_soil.

@cenlinhe cenlinhe changed the base branch from release-v4.5-WRF to release-v4.6-WRF July 17, 2024 16:59
Copy link
Collaborator

@tslin2 tslin2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@lrbison
Copy link
Author

lrbison commented Oct 18, 2024

ping on this PR, is the testing complete?

@cenlinhe
Copy link
Collaborator

ping on this PR, is the testing complete?

Sorry for the delay. Our team is overwhelmed by a number of things but this test is on top of our list and will be done soon.

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.

Error in Water Balance: "The model is losing water (ERRWAT is negative)"
4 participants