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

rick's user sample comments on PR #106

Open
miladagreen opened this issue May 3, 2024 · 0 comments
Open

rick's user sample comments on PR #106

miladagreen opened this issue May 3, 2024 · 0 comments

Comments

@miladagreen
Copy link

miladagreen commented May 3, 2024

The User Sampled branch needs a cash crop planting date (found on the Cash Crop tab), and a cover crop termination date (found on the Cover Crop tab). Note that these dates should not have default values.

The program has unspecified planting and termination dates on the Location tab. I think these can go away.

The planting date on the Location tab is actually the cover crop planting date, which isn't needed for the User Sampled branch. (I'm also unsure why it's needed in the Satellite branch, but I need to study that some more.)

@miladagreen miladagreen converted this from a draft issue May 3, 2024
@miladagreen miladagreen self-assigned this May 3, 2024
@mikahpinegar mikahpinegar moved this from 🏗 In progress to 👀 In review in PSA Planning May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Under review
Development

No branches or pull requests

1 participant