modified quality check, and gapfilling, and improved plotting #529
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
|
Hey Mitra, |
|
Thanks Alex
…On Thu, Feb 29, 2024 at 11:57 AM Alexis Renchon ***@***.***> wrote:
Hey Mitra,
You can assign reviewers on the top right of this web page.
I assigned myself (I'll have a look next week)
—
Reply to this email directly, view it on GitHub
<#529 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BCSRGXOHQHNJXPYSUA3OTCTYV5OXPAVCNFSM6AAAAABD7FEAGOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZRGU3DIOBWGI>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
run_fluxnet.jl data_tools.jl and plot_utils.jl are modified format is corrected formatted again this time by climaformat.jl formatted again this time by climaformat.jl manual formatting manual formatting manual formatting manual formatting bucket albedo parameterization cleanup changes made packages precompiled ClimaLandSimulations failed all Pkgs precompile, and code is running well kat comments on PR #527 are included md removed generated docs are-removed
AlexisRenchon
force-pushed
the
ma/fluxnet_sites
branch
from
b8a180d
to
2cd28ac
Compare
|
I squashed your commits to 1, |
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.
Purpose
This PR includes modifications for gapfilling and plotting in
run_fluxnet.jl,plot_utils.jl, anddata_tools.jl, addressing specific issues related to using longterm mean and general quality flags for gapfilling and plotting enhancements.Content
This PR introduces changes to
run_fluxnet.jl,plot_utils.jl, anddata_tools.jl. Specifically, I have:plot_utils.jlnow contains a new function plot_and_save that plots and save if needed the required model met-drivers under "results/inputs"data_tools.jlnow uses the data from same hour and same day across all years if any data is absent. If no such data is available the absent data is filled up with a coarser representative dataset as stated in "time_criteria"run_fluxnet.jlis modified as necessary and plotting output data is modified.In the Content, I have included:
Request for Review
@Espeer5 and @kmdeck, please review the changes and provide your feedback. Your insights would be invaluable in resolving the remaining issues and finalizing the PR for merge.