Skip to content

Add Setaria as genus for running BioCro#2389

Merged
tonygardella merged 3 commits intoPecanProject:developfrom
KristinaRiemer:setaria_to_biocro
Jul 29, 2019
Merged

Add Setaria as genus for running BioCro#2389
tonygardella merged 3 commits intoPecanProject:developfrom
KristinaRiemer:setaria_to_biocro

Conversation

@KristinaRiemer
Copy link
Copy Markdown
Contributor

Description

Added the genus Setaria to call_biocro

Motivation and Context

We are using PEcAn to run BioCro on Setaria viridis for a project

Review Time Estimate

  • Immediately
  • Within one week
  • When possible

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the CHANGELOG.md.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@infotroph
Copy link
Copy Markdown
Member

I assume this was considered already, but to check: Am I right that we want to eventually move away from having PEcAn-specific model parameter files in models/biocro/inst/extdata/defaults/ and therefore it's intentional that this PR doesn't add a Setaria file there?

@KristinaRiemer
Copy link
Copy Markdown
Contributor Author

Oh, I somehow missed those genus-specific files when I was searching. Thoughts @dlebauer?

@dlebauer
Copy link
Copy Markdown
Member

dlebauer commented Jul 25, 2019

good catch @infotroph yes, we will need to have a extdata/defaults/setaria.xml file. We can start by copying the sorghum.xml file.

@infotroph you are correct that we want to move away from pecan specific defaults but will wait until we start using v1.0 for that!

@KristinaRiemer
Copy link
Copy Markdown
Contributor Author

Okay, I've added that file. The only thing I changed between Sorghum and Setaria was the genus tag, should anything else be different?

@tonygardella tonygardella merged commit ac5aef2 into PecanProject:develop Jul 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants