-
Notifications
You must be signed in to change notification settings - Fork 4
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
Change cultivated aggregation logic and make sensor choice an option #165
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Has the impact on the accuracy of validation data been measured? Not sure we have data from different sensors in the validation but if we do, it's worth checking this.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #165 +/- ##
===========================================
- Coverage 82.38% 82.25% -0.14%
===========================================
Files 67 67
Lines 4951 4970 +19
===========================================
+ Hits 4079 4088 +9
- Misses 872 882 +10 ☔ View full report in Codecov by Sentry. |
I have to merge this to be able to run another continental scale and then on validation data. There would be retrace potentially. |
Changes:
any sensor says cultivated -> cultivated
toany sensor says natural -> natural
as bias towards cultivated class was observed in validationwoody
instead ofwoody_cover