Skip to content

V4 updates jjoshi #106

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 16 commits into
base: main
Choose a base branch
from
Open

V4 updates jjoshi #106

wants to merge 16 commits into from

Conversation

@stineb
Copy link
Contributor

@stineb stineb commented Mar 3, 2025

DO NOT MERGE YET.

jaideep777 and others added 16 commits July 2, 2024 21:38
@jaideep777
add downsampler and formatter for phydro drivers
a2069cf
@jaideep777
specify tidyverse namespaces
4be50f8
@jaideep777
fix IGBP types in metadata, add C3/C4 in site info
20f4e2b
@jaideep777
fix igbp types in original style; do consistency check later
4fd4ef3
@jaideep777
add legacy option to recreate pmodel forcing; bugfix note
e7617eb
@jaideep777
merge latest changes from main; related to creation of v3.3
3920745
@jaideep777
working legacy downsampling; test script with AU-ASM
283e8bd
@jaideep777
working new rsofun drivers generation
dec1cdd
@jaideep777
format_drivers now separates driver and validation data, applied QC t…
1989046 
…o val data
@jaideep777
working gapfilling; tested full drivers generation
3c2b6e1
@jaideep777
write only 24-h means in validation dataset
7c01e0e
@jaideep777
finalize batch_format rscript
5a7c053
@stineb
Merge pull request #96 from jaideep777/phydro_drivers
414438f 
New format drivers
@stineb
Merge branch 'v4_updates_jjoshi' into v4_updates_jjoshi-1
16a4a40
@stineb
Merge pull request #99 from geco-bern/v4_updates_jjoshi-1
3a6ca26 
added NEE and wind speed to drivers, interpolate temperature before L…
@fabern
Remove non-phydro versions of downsample_fluxnet_phydro() and format_…
bba8a43 
…drivers_phydro()
@stineb
Copy link
Contributor Author

stineb commented Mar 5, 2025

This contains lots of code that we may not want to merge.
@jaideep777 what was the final choice for the aggregation level used for determining growth conditions to which plants are acclimated? Did you end up using the daytime mean or a mean across 3 hr-maximum insolation hours?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@stineb @jaideep777 @fabern