Skip to content

Conversation

@kmdeck
Copy link
Member

@kmdeck kmdeck commented Oct 22, 2025

Purpose

Our default sets roughness lengths for the canopy as parameters using the toml. In almost all our example, though, we override these by using a constant times the canopy height. This PR updates the default so that it does what we want

This also creates a roughness parameterization struct which we can dispatch off of when computing surface fluxes. Since the shared canopy parameter struct then only has the earth param set, we can remove that from our code; this also enables us to change roughness paramterizations in the future

Closes #1351

To-do

Content


  • I have read and checked the items on the review checklist.

@kmdeck kmdeck force-pushed the kd/canopy_roughness branch from 16953a0 to 8916eba Compare November 14, 2025 19:00
@kmdeck kmdeck mentioned this pull request Nov 14, 2025
2 tasks
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.

Roughness lengths for canopy

2 participants