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

Canopy loss factor. #120

Merged
merged 3 commits into from
Mar 19, 2024
Merged

Canopy loss factor. #120

merged 3 commits into from
Mar 19, 2024

Conversation

drnimbusrain
Copy link
Member

@MaggieMarvin Here is an approximate canopy loss factor when bio summing option is used.

It is based on Eq. 21 of Guenther et al. 2006)

This will facilitate improved comparison of canopy-app top primary biogenic emissions from top of canopy with ground-truth flux observations (e.g., PECORINO, FLUCs, etc.) that inherently measure fluxes after chemistry and deposition loss have occurred in-canopy. We can always turn this off when directly using leaf-level canopy-app emissions with chemical box models that may have BVOC chemistry and deposition (e.g., F0AM)

@drnimbusrain drnimbusrain added enhancement New feature or request help wanted Extra attention is needed labels Feb 16, 2024
@drnimbusrain drnimbusrain marked this pull request as draft February 16, 2024 16:23
@drnimbusrain drnimbusrain linked an issue Feb 16, 2024 that may be closed by this pull request
@drnimbusrain drnimbusrain changed the title New branch for canopy loss factor. Canopy loss factor. Feb 16, 2024
@drnimbusrain drnimbusrain marked this pull request as ready for review March 1, 2024 15:20
Copy link
Collaborator

@MaggieMarvin MaggieMarvin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes work as expected - option 1 varies with canopy height and reduces emissions with loss factors on the order of 0.99 (see attached plot), while option 2 produces a constant reduction of 0.96 at all emission locations. It's good that we've added more warnings and options until we can gain some more clarity on the application of lambda to different PFTs and to species other than isoprene.
map_canopyapp_bio_NA-28

@drnimbusrain
Copy link
Member Author

@MaggieMarvin Great, thanks. I will wait to merge though, as I'd rather get historical changes in first and then resolve the conflicts only once here.

@drnimbusrain drnimbusrain merged commit 019958e into develop Mar 19, 2024
4 checks passed
@drnimbusrain drnimbusrain deleted the iso_loss branch March 19, 2024 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Biogenics] Adding Canopy Loss Factor
2 participants