Add Interception Module #286
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
hellkite500
reviewed
Mar 13, 2025
|
|
||
| ## Interception | ||
|
|
||
| The `hydrotools.events.models.interception` module includes a basic method that estimates canopy interception. The method includes reasonable defaults from the literature for average evaporation rates and trunk fractions. The primary point of entry is the `compute_interception` method which estimates canopy intercepted rainfall in units of depth. `compute_interception` requires a gross rainfall amount, an average rain rate, and a canopy fraction. The defaults assume that gross rainfall is in mm, rainfall rate in in mm/h, and canopy fraction is dimensionless. While this method can be used in different contexts, expected usage is for event-scale (single storm) catchment-wide estimates of intercepted rainfall. |
There was a problem hiding this comment.
Suggested change
| The `hydrotools.events.models.interception` module includes a basic method that estimates canopy interception. The method includes reasonable defaults from the literature for average evaporation rates and trunk fractions. The primary point of entry is the `compute_interception` method which estimates canopy intercepted rainfall in units of depth. `compute_interception` requires a gross rainfall amount, an average rain rate, and a canopy fraction. The defaults assume that gross rainfall is in mm, rainfall rate in in mm/h, and canopy fraction is dimensionless. While this method can be used in different contexts, expected usage is for event-scale (single storm) catchment-wide estimates of intercepted rainfall. | |
| The `hydrotools.events.models.interception` module includes a basic method that estimates canopy interception. The method includes reasonable defaults from the literature for average evaporation rates and trunk fractions. The primary point of entry is the `compute_interception` method which estimates canopy intercepted rainfall in units of depth. `compute_interception` requires a gross rainfall amount, an average rain rate, and a canopy fraction. The defaults assume that gross rainfall is in mm, rainfall rate is in mm/h, and canopy fraction is dimensionless. While this method can be used in different contexts, expected usage is for event-scale (single storm) catchment-wide estimates of intercepted rainfall. |
| mm (Miralles et al, 2010). Notated $S_c$. Expected to vary little by | ||
| location and leaf type (± 0.4 mm, standard deviation). Measurement | ||
| units should correspond to rainfall_rate (i.e. if rainfall_rate is in | ||
| mm/hour, then canopy_storate should be in mm). |
There was a problem hiding this comment.
Suggested change
| mm/hour, then canopy_storate should be in mm). | |
| mm/hour, then canopy_storage should be in mm). |
| rainfall_rate: float, required | ||
| Mean rainfall rate in L/T, typically mm/hour. Notated 'R'. | ||
| canopy_storage: float, required | ||
| Minimum depth of water required to saturate the canopy. Typically 1.2 |
There was a problem hiding this comment.
if this parameter has a "typical" value with little expected variance, could it be made an optional arg with 1.2 the default?
There was a problem hiding this comment.
Yeah, I chose to make these optional for the top level method, but required for the lower level methods. I figured it would be easier to maintain that way. I guess I could add another layer (module or object) to store the default values, then point all these parameters at that.
| 0.02 (Miralles et al, 2010). Notated $\\epsilon$. Unitless. Valid | ||
| range is [0.0, 1.0) | ||
| evaporation_rate: float, required | ||
| Average evaporation rate. Typically 0.3 mm/hour (Miralles et al, 2010). |
There was a problem hiding this comment.
Another one to consider for optional with a default value
There was a problem hiding this comment.
I guess the larger problem there though is dealing with unit reconciliation...
|
|
||
| Parameters | ||
| ---------- | ||
| gross_rainfall: float, require |
There was a problem hiding this comment.
Suggested change
| gross_rainfall: float, require | |
| gross_rainfall: float, required |
| mm (Miralles et al, 2010). Notated $S_c$. Expected to vary little by | ||
| location and leaf type (± 0.4 mm, standard deviation). Measurement | ||
| units should correspond to rainfall_rate (i.e. if rainfall_rate is in | ||
| mm/hour, then canopy_storate should be in mm). |
There was a problem hiding this comment.
Suggested change
| mm/hour, then canopy_storate should be in mm). | |
| mm/hour, then canopy_storage should be in mm). |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds a module to compute basic estimates of canopy interception using reasonable defaults from literature.
Additions
Removals
Changes
Testing
Notes
Todos
Checklist