Skip to content

Conversation

@dlebauer
Copy link

Getting closer!

Finishing up PecanProject#3570, I moved additional details from the original demos to the new quarto demos.

We are getting closer, @AritraDey-Dev, before merging please review to check that the updated demos:

  • have a logical sequence
  • information is clear and correct
  • content is not redundant

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Oct 15, 2025

The following sections describe the contents of these folders and files in more detail.

### Traits
Copy link
Owner

Choose a reason for hiding this comment

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

should we keep documentation on trait analysis here as we are not doing trait analysis in this workflow

The Rdata files **trait.data.Rdata** and **jagged.data.Rdata** are, respectively, the available trait data extracted from the database that was used to estimate the model parameters and that same data cleaned and formatted for the statistical code. The **list of variables that are queried is determined by what variables have priors associated with them in the definition of the PFTs**. Priors are output into **prior.distns.Rdata**. Likewise, the **list of species that are associated with a PFT determines what subset of data is extracted** out of all data matching a given variable name. Demo 3 will demonstrate how a PFT can be created or modified. To look at these files in RStudio **click on these files to load them into your workspace**. You can further examine them in the _Environment_ window or accessing them at the command line. For example, try typing ```names(trait.data)``` as this will tell you what variables were extracted, ```names(trait.data$Amax)``` will tell you the names of the columns in the Amax table, and ```summary(trait.data$Amax)``` will give you summary data about the Amax values.


### META-ANALYSIS:
Copy link
Owner

Choose a reason for hiding this comment

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

same here also,as we are not doing meta analysis in this notebook , should we keep comment on meta analysis?

Copy link
Author

Choose a reason for hiding this comment

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

Apologies for the oversight. These can be moved to the uncertainty.qmd

Copy link
Owner

Choose a reason for hiding this comment

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

even in the uncertainty notebook as we are not doing trait and meta analysis,we are just doing uncertainty analysis i.e. ensemble and sensivity analysis,can we remove that trait and meta analysis comments part from both of the notebook?


**The `/pft` directory** contains an array of outputs available that are related to the process of estimating the model parameters.

The Rdata files **trait.data.Rdata** and **jagged.data.Rdata** are, respectively, the available trait data extracted from the database that was used to estimate the model parameters and that same data cleaned and formatted for the statistical code. The **list of variables that are queried is determined by what variables have priors associated with them in the definition of the PFTs**. Priors are output into **prior.distns.Rdata**. Likewise, the **list of species that are associated with a PFT determines what subset of data is extracted** out of all data matching a given variable name. Demo 3 will demonstrate how a PFT can be created or modified. To look at these files in RStudio **click on these files to load them into your workspace**. You can further examine them in the _Environment_ window or accessing them at the command line. For example, try typing ```names(trait.data)``` as this will tell you what variables were extracted, ```names(trait.data$Amax)``` will tell you the names of the columns in the Amax table, and ```summary(trait.data$Amax)``` will give you summary data about the Amax values.
Copy link
Owner

Choose a reason for hiding this comment

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

As we are not using database in this notebook,i believe it is safe to remove this part.

Copy link
Owner

@AritraDey-Dev AritraDey-Dev left a comment

Choose a reason for hiding this comment

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

Some small changes are needed; I will make them on my end after merging.

@AritraDey-Dev AritraDey-Dev merged commit 09fa53c into AritraDey-Dev:quarto-ensemble-sensivity Oct 21, 2025
19 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants