wri / gfw-raster-analysis-lambda

GFW Raster Analysis in AWS Lambda
8 stars 4 forks source link

Allow OR filters and nested filters, add tests for carbon layers #46

Closed jterry64 closed 3 years ago

jterry64 commented 3 years ago

Pull request checklist

Please check if your PR fulfills the following requirements:

Pull request type

Please check the type of change your PR introduces:

What is the current behavior?

-Only allows single-level AND filters in query. -Multiple aggregates with no group by will return results in incorrect format.

Issue Number: GTC-1246

What is the new behavior?

Does this introduce a breaking change?

codecov[bot] commented 3 years ago

Codecov Report

Merging #46 (55773cb) into develop (a0590b1) will increase coverage by 1.37%. The diff coverage is 98.70%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #46      +/-   ##
===========================================
+ Coverage    90.35%   91.72%   +1.37%     
===========================================
  Files           20       20              
  Lines         1057     1100      +43     
===========================================
+ Hits           955     1009      +54     
+ Misses         102       91      -11     
Flag Coverage Δ
unittests 91.72% <98.70%> (+1.37%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
raster_analysis/query.py 96.11% <98.30%> (+1.62%) :arrow_up:
raster_analysis/query_executor.py 92.47% <100.00%> (+5.10%) :arrow_up:
raster_analysis/tiling.py 98.94% <100.00%> (-0.02%) :arrow_down:
raster_analysis/window.py 94.20% <100.00%> (+0.26%) :arrow_up:
tests/e2e/test_raster_analysis.py 100.00% <100.00%> (+4.20%) :arrow_up:
tests/fixtures/fixtures.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update a0590b1...55773cb. Read the comment docs.