Skip to content
Discussion options

You must be logged in to vote

Dear @huangwm29

Thank you for this post. You have spotted this mistake correctly. It should actually be written as you have suggested:
satAreaFrac = np.maximum(1 - (1 - relSat) ** self.var.arnoBeta[No], 0)

The infiltration capacity is calculated twice in CWatM:

  1. in soil.py (lines 421-444) in order to simulate the infiltration.
  2. in water_demand/irrigaiton.py (lines 142-153) in order to limit irrigation demand.

It follows that this error affects the irrigation withdrawal estimates. We have already identified this error and provided a fix in the develop branch which will be made publicly available in the next release.

I hope this makes it clear.

Best
Dor Fridman

Replies: 1 comment 1 reply

Comment options

You must be logged in to vote
1 reply
@huangwm29
Comment options

Answer selected by huangwm29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Category
Q&A
Labels
None yet
2 participants