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

Moving generate_biomee/pmodel_driver_data.R from FluxDataKit to rsofun #232

Open
marcadella opened this issue Oct 21, 2024 · 3 comments
Open

Comments

@marcadella
Copy link
Collaborator

Related to issue #228 and the fact that we are homogenizing the interfaces: should we move rsofun/data-raw/generate_biomee_driver_data.R to DataFluxKit? It looks like it is related to FluxDataKit/analysis/05_generate_demo_rsofun_drivers.R

@fabern
Copy link
Member

fabern commented Oct 21, 2024

This would also affect the generation of pmodel drivers.

The README.md states that the use https://github.com/geco-bern/FluxDataKit/blob/main/analysis/05_generate_demo_rsofun_drivers.R. However, this script has disappeared in the meantime. For the phydro branch the forcing input is anyway extended with the columns: forcing, forcing_24h, forcing_daytime, and forcing_3hrmax

  • use the phydro-updated data (generated with this script and the function fdk_format_drivers()) including the columns: forcing, forcing_24h, forcing_daytime, and forcing_3hrmax
  • also include site properties AI and MAT (according to Add site meta info FluxDataKit#100)

@stineb
Copy link
Collaborator

stineb commented Oct 21, 2024

The mentioning of rsofun/data-raw/generate_biomee_driver_data.R can be removed from FluxDataKit. The complete driver object for all sites and years is generated in FluxDataKit (analysis/02_batch_format_rsofun_drivers.R). In rsofun, let's keep the scripts that generate the demo data from that (subsetting to one site and a couple of years within that). This is done by data-raw/generate_biomee_driver_data.R and data-raw/generate_pmodel_driver_data.R. In that sense, I think there is no change needed for creating the P-model drivers. But then, the BiomeE drivers generated by rsofun/data-raw/generate_biomee_driver_data.R may be updated?

@stineb stineb closed this as completed Oct 21, 2024
@fabern fabern changed the title Moving generate_biomee_driver_data.R to FluxDataKit Moving generate_biomee/pmodel_driver_data.R from FluxDataKit to rsofun Oct 24, 2024
@fabern
Copy link
Member

fabern commented Oct 24, 2024

The way I understand it, there are the following tasks still to be done:

  • remove mentions of rsofun data generation from the FluxDataKit repository [EDIT: following this PR: https://github.com/Minor cleanup FluxDataKit#101]
  • regenerate drivers for p-model and BiomeE-model
    • generation scripts should be put: into the data-raw-folder of the repo rsofun
      • for p-model it should use analysis/b_02_batch_format_rsofun_drivers.R from the updated FDK PR
      • for BiomeE there is already a script generate_biomee_driver_data.R
    • the data-raw-folder of the repo rsofun also contains data needed for the generation
      • for BiomeE this is CH-LAE_forcing.rda
      • for p-model should we add a reduced FDK csv (containing only FR-Pue, Reichetal_Colorado, Reichetal_New_Mexico, Reichetal_Venezuela, and Reichetal_Wisconsin)?
    • updated drivers replace the *.rda-files in the data-folder of the repo rsofun

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

No branches or pull requests

3 participants