Skip to content

Conversation

@alyssa-greenwell
Copy link

@alyssa-greenwell alyssa-greenwell commented Nov 29, 2025

Added beds D-1 and D-2 to field D and changed the transplanting of some of the Ghana Iceberg Lettuce into D-1 and D-2. These changes are made to close issue #553 in the FarmData2 repository. I edited the appropriate lines in areas.csv and transplantings.csv. However when I try to build the database I am asked for a sudo fd2dev password, which I do not have and cannot find in the documentation, so the changes are yet to be tested. I require some assistance from the maintainers to either get access to the fd2dev password or to help test the changes. However being able to test this on my own would be ideal as I want to work on issue #554 next, which would require these changes be implemented.

Closes FarmData2/FarmData2#553

@alyssa-greenwell alyssa-greenwell changed the title Added Beds D-1 and D-2 feat: adds beds D-1 and D-2 to field D Nov 30, 2025
@braughtg
Copy link
Contributor

However being able to test this on my own would be ideal as I want to work on issue #554 next, which would require these changes be implemented.

These changes look good. Tag me on this PR when you have had a chance to rebuild the sample database and add it to the PR.

@alyssa-greenwell
Copy link
Author

@braughtg I rebuilt the sample database and added it to this PR. I still need to fix three failing tests on the FarmData2 repository.

@alyssa-greenwell alyssa-greenwell marked this pull request as ready for review November 30, 2025 21:13
@braughtg
Copy link
Contributor

braughtg commented Dec 1, 2025

I rebuilt the sample database and added it to this PR. I still need to fix three failing tests on the FarmData2 repository.

Great. This looks good. I will pull and test it and then merge it if everything looks good.

When you have the tests patched, make that PR to the FarmData2 upstream.

Copy link
Contributor

@braughtg braughtg left a comment

Choose a reason for hiding this comment

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

Just tested this and everything looks good! Well done @alyssa-greenwell.

I will merge this when FarmData2/FarmData2#554 is complete.

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

Successfully merging this pull request may close these issues.

SampleDB: Add beds to D and assign plants to beds.

2 participants