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

Changes the units of water flux between root and soil in ALM #1409

Merged
merged 1 commit into from
Apr 21, 2017

Conversation

bishtgautam
Copy link
Contributor

The units flux from root and soil are changed from [m H2O/s] to
[mm H2O/s], so it is consistent units of other fluxes defined in
waterflux_type.

Fixes #1371
[BFB]

The units flux from root and soil are changed from [m H2O/s] to
[mm H2O/s] to be consistent units of other fluxes defined in
waterflux_type.

Fixes #1371
jqyin added a commit that referenced this pull request Apr 19, 2017
…1409)

Changes the units of water flux between root and soil in ALM

The units flux from root and soil are changed from [m H2O/s] to
[mm H2O/s], so it is consistent units of other fluxes defined in
waterflux_type.

Fixes #1371
[BFB]
@jqyin
Copy link
Contributor

jqyin commented Apr 19, 2017

Merged to next.

@jqyin jqyin merged commit a70ce4f into master Apr 21, 2017
jqyin added a commit that referenced this pull request Apr 21, 2017
Changes the units of water flux between root and soil in ALM

The units flux from root and soil are changed from [m H2O/s] to
[mm H2O/s], so it is consistent units of other fluxes defined in
waterflux_type.

Fixes #1371
[BFB]
@jqyin
Copy link
Contributor

jqyin commented Apr 21, 2017

Merged #1409

@bishtgautam bishtgautam deleted the bishtgautam/lnd/fix-qflx-rootsoi-units branch April 24, 2017 20:50
agsalin pushed a commit that referenced this pull request May 1, 2017
Fix big mistake in scripts_regression_tests
Indent error lead to cdash always showing OK :(

Test suite: by-hand
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes #1409

User interface changes?: None

Code review: @jedwards4b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BFB PR leaves answers BFB bug fix PR Land
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants