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.

Sign up for GitHub

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

Jump to bottom

soil order not read in correctly #1793

Closed
xyk-ornl opened this issue Sep 18, 2017 · 1 comment
Closed

soil order not read in correctly #1793

xyk-ornl opened this issue Sep 18, 2017 · 1 comment
Assignees
@xyk-ornl
Labels
bug Land non-BFB PR makes roundoff changes to answers.

Comments

@xyk-ornl
Copy link
Contributor

xyk-ornl commented Sep 18, 2017 •
edited
Loading

Soil order was set as 12 instead of being read in from global surface data. This causes problems in the calculation of sorption dynamics for non-Oxisols.
@xyk-ornl xyk-ornl added bug Land non-BFB PR makes roundoff changes to answers. labels Sep 18, 2017
@xyk-ornl xyk-ornl self-assigned this Sep 18, 2017
@xyk-ornl xyk-ornl mentioned this issue Sep 18, 2017
Merged
@rljacob
Copy link
Member

rljacob commented Sep 18, 2017

What is the impact of this bug?

jqyin added a commit that referenced this issue Oct 3, 2017
@jqyin
Merge branch 'yangx/lnd/bugfix_soilorder' into next (PR #1792)
3ff1350 
Fixes the reading of soilorder

Fixes #1793

[non-BFB]
bishtgautam pushed a commit that referenced this issue Oct 4, 2017
Merge remote-tracking branch 'origin/yangx/lnd/bugfix_soilorder' into…
eaa0a1d 
… bishtgautam/lnd/testing-pr-1804 (PR #1793)
jqyin added a commit that referenced this issue Oct 6, 2017
@jqyin
Merge branch 'yangx/lnd/bugfix_soilorder' (PR #1792)
50238ac 
Fixes the reading of soilorder

Fixes #1793

[non-BFB]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xyk-ornl xyk-ornl

Labels
bug Land non-BFB PR makes roundoff changes to answers.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rljacob @xyk-ornl