Skip to content

Conversation

@grilledlambchops
Copy link
Contributor

Added rodent census 540 data. Made sure worksheets were identical and no errors were occurring.

@gmyenni
Copy link
Contributor

gmyenni commented Nov 18, 2025

2 tests are failing:

  • Not all plots entered - it looks like you forgot to add plot 9.
  • No versioning information - All commits entering new data should have [minor] in the commit message.

@grilledlambchops
Copy link
Contributor Author

For the 540 census data, it looks like plot 9 wasn't entered on the data sheet, that's why it wasn't in the excel. I added it in with code 2 for empty plot and re-committed. Is this correct? I have pushed that new commit and can create another pull request if needed.

@gmyenni
Copy link
Contributor

gmyenni commented Dec 4, 2025

That is correct. You pushed the new commit to this branch, no need to start another PR.

Comment out NDVI update section in update-data.R. 
USGS API has changed and is causing action to time out. Temporarily disabling NDVI updating until resolved.
Uncommented NDVI update process and added success message.
@gmyenni gmyenni merged commit a65f993 into main Dec 10, 2025
1 of 2 checks passed
@gmyenni gmyenni deleted the rodents_540 branch December 10, 2025 18:02
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.

3 participants