-
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
Fix stripe issue in landcover level34 #175
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #175 +/- ##
===========================================
- Coverage 81.24% 81.20% -0.04%
===========================================
Files 50 50
Lines 4532 4528 -4
===========================================
- Hits 3682 3677 -5
- Misses 850 851 +1 ☔ View full report in Codecov by Sentry. |
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.
I don't fully understand the logic but the code looks good.
odc/stats/plugins/lc_fc_wo_a0.py
Outdated
wet_clear = expr_eval( | ||
"where(b>0, _nan, a)", | ||
{"a": wet_clear, "b": raw_mask.data}, | ||
name="get_lear_pixels", |
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.
lear->clear
As the title, changes:
water_frequency
and normalise against total valid observations.water_season
from level1, use wofrequency
band in level34 insteadThe csv is updated accordingly https://dea-public-data-dev.s3-ap-southeast-2.amazonaws.com/lccs_validation/c3/data_to_plot/lccs_colour_scheme_golden_dark_au_c3.csv